-
Notifications
You must be signed in to change notification settings - Fork 200
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
chore: Migrate ACVM without merging flakes #2773
Conversation
New dependencies detected. Learn more about Socket for GitHub ↗︎
|
Releases are likely broken in this due to no release-please setup changes |
Yeah -- those changes would bump the version of Noir to 0.26, so I chose to do that separately and just have tests passing in this. Only now realising that build-wasm takes 20-30 minutes :/ |
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.
Even though releases are going to be broken, we can unblock bunch of changes and fix releases separately.
Description
Problem*
Resolves
Summary*
Documentation
This PR requires documentation updates when merged.
Additional Context
PR Checklist*
cargo fmt
on default settings.