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

Equality type failures seem to list types in wrong order #2352

Closed
ludamad opened this issue Aug 16, 2023 · 1 comment · Fixed by #2353
Closed

Equality type failures seem to list types in wrong order #2352

ludamad opened this issue Aug 16, 2023 · 1 comment · Fixed by #2353
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@ludamad
Copy link
Collaborator

ludamad commented Aug 16, 2023

Aim

Aim: Codin' up some noir

Using some snazzy equality operators
Screenshot 2023-08-16 at 5 49 36 PM

Expected Behavior

Type on left hand matches left expression
Type on right hand matches right expression

ie: Types in a binary operation should match, but found Field and Option<ValueNote>

Bug

Type on left hand matches right expression
Type on right hand matches left expression

ie: Types in a binary operation should match, but found Option<ValueNote> and Field

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

@ludamad ludamad added the bug Something isn't working label Aug 16, 2023
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Noir Aug 16, 2023
@ludamad ludamad added the good first issue Good for newcomers label Aug 16, 2023
@TomAFrench
Copy link
Member

Thanks for flagging this. This is fixed in #2353

@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in Noir Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants