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

Enforce a single Noir contract per package #2329

Closed
TomAFrench opened this issue Aug 15, 2023 · 0 comments · Fixed by #2668
Closed

Enforce a single Noir contract per package #2329

TomAFrench opened this issue Aug 15, 2023 · 0 comments · Fixed by #2668
Assignees
Labels
enhancement New feature or request

Comments

@TomAFrench
Copy link
Member

Problem

We currently support an unlimited number of contracts within a single Noir package. We've decided that we'd like there to be a single contract per package instead.

Happy Case

Multiple contracts should result in a parsing error. We can then replace the functions to pull contracts from the defmap to instead return a single contract.

As there's only a single contract we can also remove the name field from contract <name> {} however we could maintain this and simply ignore it.

Alternatives Considered

No response

Additional Context

No response

Would you like to submit a PR for this Issue?

No

Support Needs

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants