Skip to content

chore: Mark sparse_array and noir_json_parser as non-critical#9282

Merged
TomAFrench merged 1 commit intomasterfrom
ss/update-libs-criticiality
Jul 23, 2025
Merged

chore: Mark sparse_array and noir_json_parser as non-critical#9282
TomAFrench merged 1 commit intomasterfrom
ss/update-libs-criticiality

Conversation

@Savio-Sou
Copy link
Collaborator

Description

Problem*

Resolves #9281.

Summary*

Update criticality statuses of sparse_array and noir_json_parser to non-critical in https://github.com/noir-lang/noir/blob/master/EXTERNAL_NOIR_LIBRARIES.yml.

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.

@Savio-Sou Savio-Sou requested a review from TomAFrench July 22, 2025 19:43
@Savio-Sou Savio-Sou added the CI label Jul 22, 2025
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: 52a4561 Previous: 3c18e33 Ratio
test_report_AztecProtocol_aztec-packages_noir-projects_noir-protocol-circuits_crates_private-kernel-lib 2 s 1 s 2
test_report_AztecProtocol_aztec-packages_noir-projects_noir-protocol-circuits_crates_types 2 s 1 s 2

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

CC: @TomAFrench

@TomAFrench TomAFrench merged commit aac99e2 into master Jul 23, 2025
100 of 101 checks passed
@TomAFrench TomAFrench deleted the ss/update-libs-criticiality branch July 23, 2025 12:20
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 <tech@aztecprotocol.com>
Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
Co-authored-by: Jan Beneš <janbenes1234@gmail.com>
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 <tech@aztecprotocol.com>
Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
Co-authored-by: Jan Beneš <janbenes1234@gmail.com>
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 <tech@aztecprotocol.com>
Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
Co-authored-by: Jan Beneš <janbenes1234@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Relieve sparse_array's and noir_json_parser's critical library statuses

2 participants