Skip to content
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

Don't hardcode constant values #6817

Closed
Tracked by #5070
benesjan opened this issue Jun 2, 2024 · 0 comments · Fixed by #7736
Closed
Tracked by #5070

Don't hardcode constant values #6817

benesjan opened this issue Jun 2, 2024 · 0 comments · Fixed by #7736
Assignees
Labels
S-blocked Status: Blocked

Comments

@benesjan
Copy link
Contributor

benesjan commented Jun 2, 2024

Now that constants get evaluated thanks to Facundo's PR before them being generated in the TS file we can use constant propagation in Noir. This should be done to not make updating of constants pain. See this discussion for more context.

Note: Currently blocked because we don't use the new elaborator in aztec-packages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-blocked Status: Blocked
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant