Skip to content

feat: merge-train/fairies#21182

Merged
AztecBot merged 7 commits intonextfrom
merge-train/fairies
Mar 6, 2026
Merged

feat: merge-train/fairies#21182
AztecBot merged 7 commits intonextfrom
merge-train/fairies

Conversation

@AztecBot
Copy link
Collaborator

@AztecBot AztecBot commented Mar 5, 2026

BEGIN_COMMIT_OVERRIDE
fix: empty undefined offchain messages for batch calls (#21157)
feat: add aztecaddress::is_valid (#21072)
feat: add ergonomic conversions for Noir's Option<T> (#21107)
chore: use returns true for boolean fns (#21186)
feat: add salt and secret params to env.deploy (#21183)
END_COMMIT_OVERRIDE

@nventuro nventuro requested a review from LeilaWang as a code owner March 5, 2026 18:44
@AztecBot AztecBot requested a review from a team March 5, 2026 19:01
mverzilli and others added 2 commits March 5, 2026 20:36
This is necessary to make calling contract functions with Option params
relatively straightforward.
Minor inconsistency spotted by @nchamo.
@nventuro nventuro self-requested a review as a code owner March 5, 2026 20:55
nventuro and others added 3 commits March 5, 2026 22:05
Originally requested by @xorsal and @zkfrov.

This simply adds a new `deploy_opts` function which takes
`DeployOptions`, via which custom salt and secret can be speficied. In
the future we might be able to allow specifying more things.

The bigger change is that the deployment oraacle now supports salt,
secret (and deployer, currently hardcoded), so we should not need to
break this API in the near future.

Closes #16656

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Martin Verzilli <martin@aztec-labs.com>
Copy link
Collaborator

@ludamad ludamad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Auto-approved

@AztecBot AztecBot added this pull request to the merge queue Mar 6, 2026
@AztecBot
Copy link
Collaborator Author

AztecBot commented Mar 6, 2026

🤖 Auto-merge enabled after 4 hours of inactivity. This PR will be merged automatically once all checks pass.

Merged via the queue into next with commit ec124dc Mar 6, 2026
19 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants