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

Error: Cannot use circuit for type field in: ... #2181

Open
Tracked by #2162
jfecher opened this issue Aug 4, 2023 · 2 comments
Open
Tracked by #2162

Error: Cannot use circuit for type field in: ... #2181

jfecher opened this issue Aug 4, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@jfecher
Copy link
Contributor

jfecher commented Aug 4, 2023

Aim

When including the type field in the Nargo.toml, if an invalid value like "circuit" or "binary" are given, the compiler panics and issues an error message that explains the field is incorrect but does not give the possible valid values of the field.

Expected Behavior

The compiler should not panic and should suggest the possible valid values of the field

Bug

Full error message:

Error: Cannot use `circuit` for `type` field in /.../short/Nargo.toml

Location:
    crates/nargo_cli/src/cli/mod.rs:74:5
exit 1

To Reproduce

Installation Method

None

Nargo Version

No response

Additional Context

No response

Would you like to submit a PR for this Issue?

No

Support Needs

No response

@jfecher jfecher added bug Something isn't working P-MEDIUM labels Aug 4, 2023
@kevaundray
Copy link
Contributor

@phated can you take a look at this

@phated
Copy link
Contributor

phated commented Aug 7, 2023

Will do! I want to find a better way for Nargo CLI errors to be written. Something like the error message and then a longer message that contains useful suggestions like this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 📋 Backlog
Development

No branches or pull requests

4 participants