-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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 Clue language #6356
base: master
Are you sure you want to change the base?
Add Clue language #6356
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.
@Markos-Th09 Regarding your question about "unacceptable samples", the following files appear to be fixtures for testing syntax-parsing, or an example code-snippet taken from a tutorial:
The following samples are good examples of realistic, practical uses for Clue (and ideally what you should be aiming for when selecting samples):
Thanks I think we have some more of these. |
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 samples are much better, thanks!
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 can't accept this grammar without a license. Please find another grammar or license your grammar with one of these licenses and add the cached license file to this PR (run bundle exec licensed cache -c vendor/licenses/config.yml
once you've added a license file and pulled the latest SHA into this PR).
The grammar is licensed under MIT. |
Usage is really low for inclusion right now too. |
Resolved |
Description
This PR adds the clue language to linguist.
Closes #6305
Checklist:
#0009b5