-
Notifications
You must be signed in to change notification settings - Fork 740
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
Add FreeFEM lexer (inherited from C++) #1356
Conversation
@sgarnotel Sorry, this took a lot longer to get to than I originally intended! I've made some minor (mostly stylistic changes). Let me know what you think! |
@pyrmont It is ok for me. |
@sgarnotel The demo is really just to make sure that the lexer loads. It's not really intended to be used for testing of the lexing of particular constructs. If you want to do that, you can either add the construct to your visual spec and then check it visually or you can add it to the test file for the particular lexer. You can see an example of how that would work in the recent PR for the Console lexer. |
@pyrmont Ok, so all is ok for me |
Cool. I'll merge this now and it'll be in the next release of Rouge coming 14 January. |
Lexer for FreeFEM, following the recommandations of PR #1020