-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Add Leo #6970
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 Leo #6970
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.
This PR is missing the cached license file generated by this line in the add-grammar
script:
Lines 133 to 134 in 39fd5e9
log 'Caching grammar license' | |
bundle exec licensed cache -c vendor/licenses/config.yml |
Please add it.
This PR is not adding support for a single file, it's adding support for an extension which means the extension criteria applies and as with #6969, it appears quite common for a repo to contain multiple It seems quite common for |
Closes #6900 |
Might be worth adding a heuristic like |
The heuristic wouldn't have any effect as this language is the only one with the @d0cd Please merge in |
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. Thanks.
Important
The changes in this PR will not appear on GitHub until the next release has been made and deployed. See here for more details.
Thanks @lildude! We'll get this updated and merged in by Friday (earlier, if needed). |
@d0cd I need to make the release by the end of Thursday, UK time, at the latest. |
Sounds good @lildude, we'll have it in ASAP! |
It depends. If this is just updating the samples, or grammar (not needed unless it's a completely new repo as we update all grammars with each release) then this PR is fine. If it's modifying the extensions, then any new ones are likely to hold up this PR even longer. Once this has been merged, users can always use an override for any new extensions. |
Got it, we have some grammar updates and a color change but if the release process updates the submodule, then it should be fairly straightforward! |
This PR adds Linguist support for the Leo progamming language. Leo is a high-level language that allows developers to use zero-knowledge proofs in their applications while abstracting away the cryptographic complexities. Leo programs are compiled into Aleo instructions, which can be executed on the Aleo blockchain.
References
Checklist
main.leo
file is expected to occur once per repository, which meets the requirements outlined here.#6814ec
Closes #6900