Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lookbehind assertion is not fixed length #118

Closed
DecimalTurn opened this issue Nov 19, 2024 · 0 comments · Fixed by #119
Closed

Lookbehind assertion is not fixed length #118

DecimalTurn opened this issue Nov 19, 2024 · 0 comments · Fixed by #119
Labels

Comments

@DecimalTurn
Copy link
Contributor

DecimalTurn commented Nov 19, 2024

Description and Reproduction Steps

There is currently an outstanding issue with regards to github-linguist: github-linguist/linguist#3924

[ ] repository vendor/grammars/vscode-vba (from serkonda7/vscode-vba.git) (1 errors)

  • Invalid regex in grammar: source.vba (in syntaxes/vba.yaml-tmlanguage) contains a malformed regex (regex "(?i:\b(?:(?<=(Call|Function|Sub)...": lookbehind assertion is not fixed length (at offset 33))

Current Behavior

The regex rule is using a "lookbehind assertion [that] is not fixed length".

Expected Behavior

We should rewrite the rule to avoid using lookbehind by using the "begin/beginCaptures" syntax.

Additional Information

No response

Environment details

NA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants