-
Notifications
You must be signed in to change notification settings - Fork 612
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 Dart support #3973
Add Dart support #3973
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM from little review and testing apart from:
- the comment about the patch file
- the absence of tags parsing. the custom filetype version uses Java, which seems to do a decent job (beware that not knowing the language, I just copied a couple of samples and lanced at the results), should this one do the same?
This is what I wanted to test but then forgot about it. I'll add it. |
This PR adds support for the Dart language: 1. It uses the Dart lexer which has been recently merged to lexilla 2. The keywords are based on Notepad4 keywords 3. Build menu settings is based on geany#371
@b4n I've rebased this PR on top of master, resolved conflicts and updated build commands based on #371 (comment) (I didn't receive any feedback there but I think it can be easily modified if users report problems with them). Is it alright to merge this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGBI and compared to previous version should also work fine 👍
This PR adds support for the Dart language: