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

ENG-209 Create ETH Verifier Contracts R*M #37

Closed
wants to merge 14 commits into from

Conversation

0xmovses
Copy link
Contributor

@0xmovses 0xmovses commented Apr 24, 2024

See full ticket: https://movement.atlassian.net/browse/ENG-5?atlOrigin=eyJpIjoiNGM2ZmIxOTQyMDBhNDlkOGJmZTdmYTIxY2Q1NzBjMmYiLCJwIjoiaiJ9

The purpose of this PR was to implement the design laid out in the RFC with the Risc0 Groth16 contract primitives, and to make sure these contracts build.

cd protocol-units/dispute && forge build

Unit tests are going to be added in the next PR, where any implementation details can be ironed out/ improved on.

@0xmovses 0xmovses changed the base branch from main to monza April 24, 2024 14:53
@0xmovses 0xmovses changed the title ENG-189 Setup Verifier Contracts ENG-189 / ENG-209 Setup & Create Verifier Contracts Apr 29, 2024
@0xmovses 0xmovses changed the title ENG-189 / ENG-209 Setup & Create Verifier Contracts ENG-189 / ENG-209 Setup & Create Verifier Contracts R*M Apr 30, 2024
@0xmovses 0xmovses self-assigned this Apr 30, 2024
@0xmovses 0xmovses changed the title ENG-189 / ENG-209 Setup & Create Verifier Contracts R*M ENG-209 Create Verifier Contracts R*M Apr 30, 2024
@0xmovses 0xmovses requested a review from l-monninger April 30, 2024 00:05
@0xmovses 0xmovses changed the title ENG-209 Create Verifier Contracts R*M ENG-209 Create ETH Verifier Contracts R*M Apr 30, 2024
@0xmovses 0xmovses changed the base branch from monza to main May 1, 2024 01:35
@l-monninger
Copy link
Collaborator

@0xmovses So, #42 effectively extends this, right?

@l-monninger l-monninger closed this May 6, 2024
@0xmovses
Copy link
Contributor Author

0xmovses commented May 6, 2024

Yes that’s right

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants