build: update cross-repo angular dependencies#32694
Conversation
alan-agius4
added
action: review
angular-robot
bot
added
the
area: build & ci
alan-agius4
added
target: major
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates various Angular dependencies across the repository, primarily to move to version 22. The changes involve bumping versions in package.json files, constants.bzl, and MODULE.bazel, with corresponding updates to pnpm-lock.yaml. My review focused on the consistency of these version updates. I found one potential issue in constants.bzl where a peer dependency version for ng-packagr appears to have been missed during the update to version 22. Otherwise, the dependency updates look correct and consistent with the goal of the pull request.
alan-agius4
force-pushed
the
pr-takeover-32681
branch
from
41be957 to
443791f
Compare
There was a problem hiding this comment.
Code Review
This pull request updates various Angular-related dependencies across the repository. The changes include updating commit hashes for Bazel modules, bumping version constants for Angular framework and ng-packagr, and updating devDependencies and peerDependencies in several package.json files. The lockfile (pnpm-lock.yaml) has been updated accordingly to reflect these changes. The updates are consistent and appear to be a standard dependency bump.
alan-agius4
force-pushed
the
pr-takeover-32681
branch
2 times, most recently
from
e12882c to
beffd21
Compare
See associated pull request for more information. Closes angular#32681 as a pr takeover
alan-agius4
force-pushed
the
pr-takeover-32681
branch
from
beffd21 to
058f761
Compare
alan-agius4
added
action: merge
|
This PR was merged into the repository. The changes were merged into the following branches:
|
2 participants
See associated pull request for more information.
Closes #32681 as a pr takeover