-
Notifications
You must be signed in to change notification settings - Fork 225
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
Change all tests to use assert
instead of constrain
#1251
Comments
@makluganteng This is a good first issue, if you are interested :) |
Yup i can take this task |
@kevaundray Hi, i kinda need help on setting up the env so that i could test it out. I cant seem to get in the discord either. |
@makluganteng which point are you stuck at? |
when im installing the direnv. nix --extra-experimental-features install nixpkgs#direnv |
Hi @makluganteng , Thank you for the willingness to take up this task -- I'm going to re-assign this issue to someone on the team. If you are still having installation problems, please open up an issue for it and flag it on our discourse: https://discourse.aztec.network/c/noir/7 |
Problem
Now that we are moving towards
assert
instead ofconstrain
. We should modify our examples to useassert
.The examples live here
Proposed solution
In each example, rename
constrain expr
to beassert(expr)
.Note: There is an
assert
example which uses theassert
keyword instead of `constrain.Alternatives considered
No response
Additional context
No response
Submission Checklist
The text was updated successfully, but these errors were encountered: