Skip to content

ci: fix angular snapshot setup script for pnpm v12 - #33959

Merged
crisbeto merged 1 commit into
angular:mainfrom
josephperrott:fix/scheduled-ci-snapshot-setup
Oct 15, 2026
Merged

crisbeto merged 1 commit into
angular:mainfrom
josephperrott:fix/scheduled-ci-snapshot-setup

Conversation

@josephperrott

Copy link
Copy Markdown
Member

Fixes the Setting up Angular snapshot builds step in scheduled-ci.yml:

  • Removes the duplicate angular/ prefix in buildsUrl (repoName already includes angular/), so snapshot dependencies point to github:angular/<pkg>-builds#main instead of github:angular/angular/<pkg>-builds#main.
  • Writes overrides (using plain package-name keys instead of Yarn's **/<pkg> selector) and blockExoticSubdeps: false to pnpm-workspace.yaml instead of package.json, matching the pnpm v11/v12 workspace configuration.

Fixes the doubled `angular/` repository prefix in snapshot build URLs and updates `setup-angular-snapshots.js` to write `overrides` and `blockExoticSubdeps: false` to `pnpm-workspace.yaml` instead of `package.json`.
@angular-robot angular-robot Bot added the area: build & ci Related the build and CI infrastructure of the project label Oct 14, 2026
@josephperrott josephperrott added target: patch This PR is targeted for the next patch release action: merge The PR is ready for merge by the caretaker labels Oct 14, 2026
@crisbeto
crisbeto merged commit 0bc28f0 into angular:main Oct 15, 2026
35 checks passed
crisbeto pushed a commit that referenced this pull request Oct 15, 2026
Fixes the doubled `angular/` repository prefix in snapshot build URLs and updates `setup-angular-snapshots.js` to write `overrides` and `blockExoticSubdeps: false` to `pnpm-workspace.yaml` instead of `package.json`.

(cherry picked from commit 0bc28f0)
@crisbeto

Copy link
Copy Markdown
Member

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

@josephperrott
josephperrott deleted the fix/scheduled-ci-snapshot-setup branch October 15, 2026 15:22
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: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants