Skip to content

build: update cross-repo angular dependencies (22.0.x)#33173

Merged
alan-agius4 merged 1 commit into
angular:22.0.xfrom
angular-robot:ng-renovate/22.0.x-cross-repo-angular-dependencies
May 13, 2026
Merged

build: update cross-repo angular dependencies (22.0.x)#33173
alan-agius4 merged 1 commit into
angular:22.0.xfrom
angular-robot:ng-renovate/22.0.x-cross-repo-angular-dependencies

Conversation

@angular-robot
Copy link
Copy Markdown
Contributor

@angular-robot angular-robot commented May 12, 2026

This PR contains the following updates:

Package Type Update Change
@angular/ng-dev devDependencies digest f084e2e2c36222
devinfra git_override digest f91b383eaa9aaa
rules_angular git_override digest 19914e2045f984

🔡 If you wish to disable git hash updates, add ":disableDigestUpdates" to the extends array in your config.


  • If you want to rebase/retry this PR, check this box

@angular-robot angular-robot added action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: automation This PR is targeted to only merge into the branch defined in Github [bot use only] labels May 12, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates several Bazel module dependencies and project package versions, including Angular packages and build tools. A review comment highlights an inconsistency in the MODULE.bazel.lock file, where partial hash definitions for older versions remain, and suggests running bazel mod tidy to ensure the lock file is correctly synchronized.

Comment thread MODULE.bazel.lock
Comment on lines +27 to +28
"https://bcr.bazel.build/modules/aspect_rules_js/3.1.1/MODULE.bazel": "b83cf3ee44837345f1c926d70b96453deb5e244de43d08dcd7acad8d381c275a",
"https://bcr.bazel.build/modules/aspect_rules_js/3.1.1/source.json": "2806c2d7ce5993f68b74df5f3e2de45d4b2a5798afedd459d88e37c75562da97",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The update to aspect_rules_js (and similarly for aspect_rules_ts, package_metadata, and platforms) appears inconsistent in the registryFileHashes section. While hashes for version 3.1.1 were added, the MODULE.bazel hash for the old version 3.0.3 was retained (line 26) while its source.json hash was removed. This results in an invalid state for the lock file where a version is partially defined. It is recommended to run bazel mod tidy to ensure the lock file is correctly synchronized and cleaned of unused version hashes.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates various project dependencies and Bazel module configurations. Key changes include bumping Angular-related packages to version 22.0.0-next.12, updating ng-packagr to 22.0.0-next.4, and refreshing several Bazel rules and toolchain versions within MODULE.bazel and MODULE.bazel.lock. There were no review comments provided, so I have no feedback to offer.

@angular-robot angular-robot force-pushed the ng-renovate/22.0.x-cross-repo-angular-dependencies branch 3 times, most recently from 14860f1 to 1f65d3a Compare May 13, 2026 13:07
See associated pull request for more information.
@angular-robot angular-robot force-pushed the ng-renovate/22.0.x-cross-repo-angular-dependencies branch from 1f65d3a to e6f0473 Compare May 13, 2026 16:38
@alan-agius4 alan-agius4 merged commit a709b42 into angular:22.0.x May 13, 2026
34 checks passed
@alan-agius4
Copy link
Copy Markdown
Collaborator

This PR was merged into the repository. The changes were merged into the following branches:

@alan-agius4 alan-agius4 deleted the ng-renovate/22.0.x-cross-repo-angular-dependencies branch May 13, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: automation This PR is targeted to only merge into the branch defined in Github [bot use only]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants