Skip to content

Conversation

@vezenovm
Copy link
Contributor

@vezenovm vezenovm commented Jul 23, 2025

Description

Problem*

Resolves failiing checks for noir-lang/sparse_array:

Run .github/scripts/check_test_results.sh .github/critical_libraries_status/noir-lang/sparse_array/.failures.jsonl .github/critical_libraries_status/noir-lang/sparse_array/.actual.jsonl
Error: expected external library not to compile, but it did.
You could create '.github/critical_libraries_status/noir-lang/sparse_array/.failures.jsonl' with these contents:

Summary*

sparse_array works post noir-lang/sparse_array#20

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.

@vezenovm vezenovm requested a review from TomAFrench July 23, 2025 12:29
@TomAFrench TomAFrench merged commit ad16599 into master Jul 23, 2025
36 checks passed
@TomAFrench TomAFrench deleted the mv/sparse-array-compiles-now branch July 23, 2025 12:31
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Test Suite Duration'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite Current: 0289561 Previous: aac99e2 Ratio
test_report_zkpassport_noir_rsa_ 2 s 1 s 2

This comment was automatically generated by workflow using github-action-benchmark.

CC: @TomAFrench

github-merge-queue bot pushed a commit to AztecProtocol/aztec-packages that referenced this pull request Jul 30, 2025
Syncs Noir up to commit `b28776c8a68dbc319e2ffcee350b8e2a71b20096` and
includes this commits:

chore: Document type coercions
(noir-lang/noir#9292)
feat: add better handling for follow-on optimizations in `simplify_cfg`
(noir-lang/noir#9276)
chore: Update noir-lang/sparse_array failures status
(noir-lang/noir#9289)
chore: Mark `sparse_array` and `noir_json_parser` as non-critical
(noir-lang/noir#9282)
fix: Use `u32` in `RepeatedValue` example in generics docs
(noir-lang/noir#9273)
chore!: enforce visibility warnings as errors
(noir-lang/noir#7657)

At commit `60222bb380da2c6840adb91317bee2ca4db30d2a` the Noir sync
beyond `b28776c8a68dbc319e2ffcee350b8e2a71b20096`

---------

Co-authored-by: AztecBot <[email protected]>
Co-authored-by: Tom French <[email protected]>
Co-authored-by: Jan Beneš <[email protected]>
github-merge-queue bot pushed a commit to AztecProtocol/aztec-packages that referenced this pull request Jul 30, 2025
Syncs Noir up to commit `b28776c8a68dbc319e2ffcee350b8e2a71b20096` and
includes this commits:

chore: Document type coercions
(noir-lang/noir#9292)
feat: add better handling for follow-on optimizations in `simplify_cfg`
(noir-lang/noir#9276)
chore: Update noir-lang/sparse_array failures status
(noir-lang/noir#9289)
chore: Mark `sparse_array` and `noir_json_parser` as non-critical
(noir-lang/noir#9282)
fix: Use `u32` in `RepeatedValue` example in generics docs
(noir-lang/noir#9273)
chore!: enforce visibility warnings as errors
(noir-lang/noir#7657)

Commit `60222bb380da2c6840adb91317bee2ca4db30d2a` causes failing tests
so could not sync Noir beyond `b28776c8a68dbc319e2ffcee350b8e2a71b20096`
(relevant slack discussion
[here](https://aztecprotocol.slack.com/archives/C0183F0V42V/p1753811323086949)).

---------

Co-authored-by: AztecBot <[email protected]>
Co-authored-by: Tom French <[email protected]>
Co-authored-by: Jan Beneš <[email protected]>
github-merge-queue bot pushed a commit to AztecProtocol/aztec-packages that referenced this pull request Jul 30, 2025
Syncs Noir up to commit `b28776c8a68dbc319e2ffcee350b8e2a71b20096` and
includes this commits:

chore: Document type coercions
(noir-lang/noir#9292)
feat: add better handling for follow-on optimizations in `simplify_cfg`
(noir-lang/noir#9276)
chore: Update noir-lang/sparse_array failures status
(noir-lang/noir#9289)
chore: Mark `sparse_array` and `noir_json_parser` as non-critical
(noir-lang/noir#9282)
fix: Use `u32` in `RepeatedValue` example in generics docs
(noir-lang/noir#9273)
chore!: enforce visibility warnings as errors
(noir-lang/noir#7657)

Commit `60222bb380da2c6840adb91317bee2ca4db30d2a` causes failing tests
so could not sync Noir beyond `b28776c8a68dbc319e2ffcee350b8e2a71b20096`
(relevant slack discussion
[here](https://aztecprotocol.slack.com/archives/C0183F0V42V/p1753811323086949)).

---------

Co-authored-by: AztecBot <[email protected]>
Co-authored-by: Tom French <[email protected]>
Co-authored-by: Jan Beneš <[email protected]>
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.

3 participants