-
Notifications
You must be signed in to change notification settings - Fork 527
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
Rename constructor Proof_level.{None -> No_check}
#16105
base: compatible
Are you sure you want to change the base?
Conversation
!ci-nightly-me |
!ci-build-me |
a1df77c
to
3531efc
Compare
!ci-build-me |
i believe the nightly is failing because the container name i am referencing this job https://buildkite.com/o-1-labs-2/mina-end-to-end-nightlies/builds/2557 (which doesn't reference this commit but has similiar changes) |
I have attempted to fix: #16106, and will re-run the nightly on that branch. |
that branch passes the nightly https://buildkite.com/o-1-labs-2/mina-end-to-end-nightlies/builds/2563#_ I do think it took some massaging. The following observations were made:
|
Using widely used None for name of constructor lead to an issue when ~proof_level parameter changed its type and compiler left it unnoticed.
3531efc
to
4187785
Compare
…no_check lightnet tagging to proof level none fix
Branch includes a lightnet fix to make nightly pass |
!ci-build-me |
!ci-nightly-me |
Using widely used
None
for name of constructor lead to an issue when~proof_level
parameter changed its type and compiler left it unnoticed.Explain how you tested your changes:
Checklist: