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

// in string breaks highlighting #1

Closed
varon opened this issue Jul 18, 2023 · 4 comments
Closed

// in string breaks highlighting #1

varon opened this issue Jul 18, 2023 · 4 comments

Comments

@varon
Copy link
Contributor

varon commented Jul 18, 2023

Thank you for continuing this project!

Simple bug report:

let howToBreakTheHighlighting = "//"

Put that line anywhere in your code to instantly break all following highlighting for the rest of the file.

This is a pretty show stopping issue because any file with a URL (i.e. https:// breaks)

@kaashyapan
Copy link
Owner

Hi

I edited the readme.

Please continue to report bugs in the original repo. I think the maintainers are still responding.
I intend to put some effort on improving this project and will look into fixing this bug as well.

@varon
Copy link
Contributor Author

varon commented Jul 19, 2023

Understood. Given the disconnected fork + reset commit history (why was this done?), that was not an expected answer.

Also, as this fork is used in helix-editor - https://github.com/helix-editor/helix/pull/7619/files#diff-f3ea26d68561619ec2cb9ee551ef03916b667bd5f55b2ae2c54cae96e8821b2eR2678, it seemed to make the most sense to add that here.

@TheJayMann
Copy link

Given that this has been fixed upstream, that upstream currently does not publish parser.c as is required by helix editor, and that helix editor is currently configured to use this repository, can we have this repository pull in the new changes and update the generated files so that helix can be configured to pull in the grammar with this issue fixed, hopefully before the next release, which, based on the frequency of previous releases, is likely soon?

@kaashyapan
Copy link
Owner

Merged
helix-editor/helix#8024

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

No branches or pull requests

3 participants