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.

Sign up for GitHub

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

Jump to bottom

Document new opcodes. #40

Merged
merged 3 commits into from
Oct 18, 2023
Merged

Document new opcodes. #40

merged 3 commits into from
Oct 18, 2023

Conversation

acoglio
Copy link
Contributor

@acoglio acoglio commented Sep 22, 2023

Also minor improvments to existing text.

@acoglio
Document new opcodes.
46522de 
Also minor improvments to existing text.
bendyarm
bendyarm reviewed Sep 22, 2023
View reviewed changes
arc-0002/README.md Outdated Show resolved Hide resolved
bendyarm
bendyarm reviewed Sep 22, 2023
View reviewed changes
arc-0002/README.md Outdated Show resolved Hide resolved
acoglio and others added 2 commits September 21, 2023 22:12
@acoglio @bendyarm
Update arc-0002/README.md
7d017b7 
Co-authored-by: Eric McCarthy <[email protected]>
Signed-off-by: Alessandro Coglio <[email protected]>
@acoglio @bendyarm
Update arc-0002/README.md
1ad2ed7 
Co-authored-by: Eric McCarthy <[email protected]>
Signed-off-by: Alessandro Coglio <[email protected]>
@acoglio acoglio requested a review from bendyarm September 22, 2023 05:13
bendyarm
bendyarm approved these changes Sep 22, 2023
View reviewed changes
Copy link
Contributor

@bendyarm bendyarm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Keeps a consistent feel with the type names
(although we might want to change them to match case
with the type names used in Aleo instructions code, that
would be more pervasive).

@collinc97 collinc97 merged commit a5a8d8b into master Oct 18, 2023
2 checks passed
@collinc97 collinc97 deleted the feat/new-opcodes branch October 18, 2023 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@collinc97 collinc97 collinc97 approved these changes

@bendyarm bendyarm bendyarm approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@acoglio @bendyarm @collinc97