Skip to content
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 Aleo instructions #6969

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

d0cd
Copy link

@d0cd d0cd commented Jul 26, 2024

This PR adds Linguist support for Aleo instructions, the format recognized and executed by the Aleo Virtual Machine (AVM). The AVM is a virtual machine with built-in cryptographic primitives for creating and verifying zero-knowledge proofs.

References

Checklist

@d0cd d0cd requested a review from a team as a code owner July 26, 2024 20:55
@d0cd d0cd mentioned this pull request Jul 26, 2024
5 tasks
@lildude lildude changed the title Add Linguist support for Aleo instructions. Add Aleo instructions Aug 5, 2024
@d0cd
Copy link
Author

d0cd commented Aug 14, 2024

Hi @lildude! Do you have any feedback on this PR? Happy to incorporate any suggestions to make it more reviewable.

@lildude
Copy link
Member

lildude commented Aug 14, 2024

The extension of the new language is used in hundreds of repositories on GitHub.com. https://github.com/search?q=path%3A*.aleo+aleo+NOT+owner%3AProvableHQ++NOT+owner%3AAleoNet+NOT+is%3Afork&type=code. Note that a file is expected to occur once per repository, which meets the requirements outlined #5756.

That restriction is for cases of only one file per repo. A quick look at a few of the search results shows it's very common for repos to contain more than one .aleo file, commonly in an imports directory.

This means things are a long way from reaching the usage requirements.

@lildude lildude mentioned this pull request Aug 14, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants