Skip to content

build: update cross-repo angular dependencies#33179

Merged
alan-agius4 merged 1 commit into
angular:22.0.xfrom
alan-agius4:22-ci-fix
May 13, 2026
Merged

build: update cross-repo angular dependencies#33179
alan-agius4 merged 1 commit into
angular:22.0.xfrom
alan-agius4:22-ci-fix

Conversation

@alan-agius4
Copy link
Copy Markdown
Collaborator

See associated pull request for more information.

Closes #33118 as a pr takeover

See associated pull request for more information.

Closes angular#33118 as a pr takeover
@angular-robot angular-robot Bot added the area: build & ci Related the build and CI infrastructure of the project label May 13, 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 primarily focuses on updating numerous dependencies and Bazel module overrides to their latest versions across the workspace. Key changes include disabling chunk optimization for server builds in the application builder to prevent incorrect preloading and setting allowedHosts to a wildcard in the SSR CommonEngine as validation is handled elsewhere. Feedback was provided to improve the clarity of a TODO comment by using the term 'check' instead of 'log' and to use a destructured variable for consistency. Additionally, a minor grammatical correction was suggested for a comment in the SSR engine.

Comment thread packages/angular/build/src/builders/application/execute-build.ts
Comment thread packages/angular/ssr/node/src/common-engine/common-engine.ts
@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: rc This PR is targeted for the next release-candidate labels May 13, 2026
@alan-agius4 alan-agius4 requested review from clydin and dgp1130 May 13, 2026 08:19
@alan-agius4 alan-agius4 removed the request for review from clydin May 13, 2026 16:08
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels May 13, 2026
@alan-agius4 alan-agius4 merged commit daca934 into angular:22.0.x May 13, 2026
48 checks passed
@alan-agius4 alan-agius4 deleted the 22-ci-fix branch May 13, 2026 16:09
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: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants