-
Notifications
You must be signed in to change notification settings - Fork 589
A first draft for simple syntax highlighting #624
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
base: main
Are you sure you want to change the base?
Conversation
|
looking forward to the other languages. they seemed to work well enough in https://github.com/microsoft/edit/tree/4f36e2afe2a84ed339829ea6c63242fb9f4b7de3 (well, really long Here Strings made the rest of the file red, but ones with a reasonable length were fine) |
|
PR awaiting 2 pending checks still. |
|
Working on this feature? In the current economy?! |
|
Thanks for your work on this. I've played around with this branch a bit and want to share my thoughts on the current implementation. I tried to hack together basic C highlighting and noticed there is some state missing for "constants" (convention with all-uppercase variables usually defined with Could the regexes be loaded from a file in the future? It might save some work to implement the basic functionality now by parsing a hardcoded string rather than defining everything in Another thing I wondered about is why the Edit: |
|
FYI I've continued development in the https://github.com/microsoft/edit/tree/dev/lhecker/syntax-highlighting-alt branch where I've since written a custom language with proper compiler. It even has rudimentary support for variables. It's fairly time consuming to develop so it's taking a while. It's also certainly not done yet, but please feel free to check it out already. You can find the definition files under |


This adds syntax highlighting for git commit and rebase messages.
Support for other languages is on hold until I finished writing a
proper VM for executing the highlighting instructions.
Closes #18