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

chore: add call formatter #3102

Merged
merged 10 commits into from
Oct 12, 2023
Merged

chore: add call formatter #3102

merged 10 commits into from
Oct 12, 2023

Conversation

jonybur
Copy link
Contributor

@jonybur jonybur commented Oct 11, 2023

Description

Problem*

Resolves

Summary*

Documentation

  • This PR requires documentation updates when merged.

    • I will submit a noir-lang/docs PR.
    • I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

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

@jonybur jonybur changed the title chore: add call expression, with test chore: add call expressions Oct 11, 2023
@jfecher
Copy link
Contributor

jfecher commented Oct 11, 2023

Can you run cargo fmt?

jfecher
jfecher previously approved these changes Oct 11, 2023
Copy link
Contributor

@jfecher jfecher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

tooling/nargo_fmt/src/visitor/expr.rs Outdated Show resolved Hide resolved
@TomAFrench TomAFrench changed the title chore: add call expressions chore: add support for call expressions in formatter Oct 11, 2023
TomAFrench
TomAFrench previously approved these changes Oct 11, 2023
@TomAFrench
Copy link
Member

TomAFrench commented Oct 11, 2023

Oof, what did I do? Phew, was already broken

TomAFrench
TomAFrench previously approved these changes Oct 11, 2023
@jonybur jonybur changed the title chore: add support for call expressions in formatter chore: add call formatter Oct 11, 2023
@jfecher jfecher added this pull request to the merge queue Oct 12, 2023
Merged via the queue into master with commit 526a68e Oct 12, 2023
27 checks passed
@jfecher jfecher deleted the jb-call-expression branch October 12, 2023 09:48
TomAFrench added a commit that referenced this pull request Oct 12, 2023
* master:
  chore: clean up JS dependencies (#3114)
  chore: add call formatter (#3102)
  chore: remove unnecessary clone when executing brillig (#3120)
  feat: return compilation errors from noir_wasm (#3091)
  feat: Save Brillig execution state in ACVM (#3026)
  chore: add method call formatter (#3106)
  feat!: Pass ACIR to ACVM by reference rather than passing ownership (#2872)
  chore: add member access formatter (#3109)
  fix: Determinism of fallback transformer (#3100)
  chore: clippy (#3108)
  feat: adding destroy and options object (#3105)
  chore: standardize witness map serialization format in JS (#3104)
  chore: add support for index expressions in formatter (#3069)
  feat: Implement impl specialization (#3087)
  chore: Update ACIR artifacts (#3086)
TomAFrench added a commit that referenced this pull request Oct 12, 2023
TomAFrench added a commit that referenced this pull request Oct 12, 2023
* master: (29 commits)
  fix: minor problems with `aztec` publishing (#3095)
  chore: clean up JS dependencies (#3114)
  chore: add call formatter (#3102)
  chore: remove unnecessary clone when executing brillig (#3120)
  feat: return compilation errors from noir_wasm (#3091)
  feat: Save Brillig execution state in ACVM (#3026)
  chore: add method call formatter (#3106)
  feat!: Pass ACIR to ACVM by reference rather than passing ownership (#2872)
  chore: add member access formatter (#3109)
  fix: Determinism of fallback transformer (#3100)
  chore: clippy (#3108)
  feat: adding destroy and options object (#3105)
  chore: standardize witness map serialization format in JS (#3104)
  chore: add support for index expressions in formatter (#3069)
  feat: Implement impl specialization (#3087)
  chore: Update ACIR artifacts (#3086)
  feat: Implement automatic dereferencing for indexing lvalues (#3083)
  fix: disallow returning constant values (#2978)
  feat: Implement automatic dereferencing for index expressions (#3082)
  feat: add `execute` method to `Noir` class (#3081)
  ...
TomAFrench added a commit that referenced this pull request Oct 12, 2023
* master:
  feat: Optimize euclidean division acir-gen (#3121)
  chore: simplify `AcirVarData` where possible in multiplication (#3124)
  feat: Remove unnecessary truncation of boolean multiplication (#3122)
  chore: use `circuit.get_assert_message` instead of closure/helper function (#3127)
  fix: minor problems with `aztec` publishing (#3095)
  chore: clean up JS dependencies (#3114)
  chore: add call formatter (#3102)
  chore: remove unnecessary clone when executing brillig (#3120)
  feat: return compilation errors from noir_wasm (#3091)
Sakapoi pushed a commit to Sakapoi/noir_fork that referenced this pull request Oct 19, 2023
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.

4 participants