-
Notifications
You must be signed in to change notification settings - Fork 159
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
feat: Check initialization arguments in constructors #5144
Merged
Merged
Commits on Mar 13, 2024
-
feat: Check init args match address preimage
The responsibility to check that the initialization arguments used when calling the initializer as correct is responsibility of the initializer itself. Without this check, a deployer could commit to a set of init args in the address preimage, but then call the initializer using something totally different.
Configuration menu - View commit details
-
Copy full SHA for fd16529 - Browse repository at this point
Copy the full SHA fd16529View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0fcade3 - Browse repository at this point
Copy the full SHA 0fcade3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 84be5c4 - Browse repository at this point
Copy the full SHA 84be5c4View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3e041c8 - Browse repository at this point
Copy the full SHA 3e041c8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 03ef986 - Browse repository at this point
Copy the full SHA 03ef986View commit details -
Configuration menu - View commit details
-
Copy full SHA for 02df977 - Browse repository at this point
Copy the full SHA 02df977View commit details -
Configuration menu - View commit details
-
Copy full SHA for dfa1974 - Browse repository at this point
Copy the full SHA dfa1974View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7549ba1 - Browse repository at this point
Copy the full SHA 7549ba1View commit details -
Configuration menu - View commit details
-
Copy full SHA for c64e4f1 - Browse repository at this point
Copy the full SHA c64e4f1View commit details -
Configuration menu - View commit details
-
Copy full SHA for 37b4af3 - Browse repository at this point
Copy the full SHA 37b4af3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 90a4f3c - Browse repository at this point
Copy the full SHA 90a4f3cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 235d22d - Browse repository at this point
Copy the full SHA 235d22dView commit details -
Update noir-projects/aztec-nr/aztec/src/initializer.nr
Co-authored-by: Alex Gherghisan <alexghr@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for ecfead7 - Browse repository at this point
Copy the full SHA ecfead7View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.