-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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 LigoLANG #5811
Add LigoLANG #5811
Conversation
Not sure about the License. I generate it with the script but I pull this from my grammar repo. I think you need the license of the language right? |
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.
We're going to need a sample for each extension you're adding support for.
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.
These languages are syntactically similar enough to their namesakes that we can afford to reuse their CodeMirror and ACE modes.
Co-authored-by: Nixinova <nixinova@gmail.com>
Co-authored-by: Nixinova <nixinova@gmail.com>
Co-authored-by: Nixinova <nixinova@gmail.com>
Co-authored-by: John Gardner <gardnerjohng@gmail.com>
Co-authored-by: John Gardner <gardnerjohng@gmail.com>
Co-authored-by: John Gardner <gardnerjohng@gmail.com>
Thanks for the reviews
I have added samples for the other two languages.
thanks, I committed your suggestions :) |
Looks like you've got some test failures to address too. |
you are right. It should be fix now |
Ok. We should be ready to merged |
Description
Checklist: