Skip to content

Conversation

forkfury
Copy link

Description

Problem*

Multiple code segments used redundant comparisons like == true or == false, which clutter readability and go against idiomatic style.

Summary*

  • Replaced == true and == false with direct boolean usage (e.g., if flag == true → if flag).

  • Updated several assert(...) == true statements to cleaner assert(...).

  • Simplified conditionals across Noir standard library and test programs.

  • Improved error messaging and removed redundant logic in a JS preprocessor.

Additional Context

Documentation*

Check one:

  • No documentation needed.
  • Documentation included in this PR.
  • [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

Copy link
Contributor

Thank you for your contribution to the Noir language.

Please do not force push to this branch after the Noir team have started review of this PR. Doing so will only delay us merging your PR as we will need to start the review process from scratch.

Thanks for your understanding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant