Skip to content

build: update pnpm to v12 - #34038

Merged
alan-agius4 merged 1 commit into
angular:mainfrom
alan-agius4:pr-takeover-34029
Sep 8, 2026
Merged

build: update pnpm to v12#34038
alan-agius4 merged 1 commit into
angular:mainfrom
alan-agius4:pr-takeover-34029

Conversation

@alan-agius4

Copy link
Copy Markdown
Collaborator

See associated pull request for more information.

PR Checklist

Please check to confirm your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

See associated pull request for more information.
@angular-robot angular-robot Bot added the area: build & ci Related the build and CI infrastructure of the project label Sep 8, 2026
@alan-agius4 alan-agius4 added target: minor This PR is targeted for the next minor release action: review The PR is still awaiting reviews from at least one requested reviewer and removed area: build & ci Related the build and CI infrastructure of the project labels Sep 8, 2026
@alan-agius4
alan-agius4 requested a review from clydin September 8, 2026 06:56

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

Copy link
Copy Markdown

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 upgrades the pnpm version from 11.25.0 to 12.3.4, replaces the shorthand -s flag with --silent across various configuration files and scripts, and removes the deprecated confirmModulesPurge option from pnpm-workspace.yaml. The feedback recommends using a semver range (e.g., ^12.3.4) for the engines.pnpm field instead of a strict exact version to avoid blocking developers with minor or patch updates of pnpm v12.

Comment thread package.json
@alan-agius4 alan-agius4 closed this Sep 8, 2026
@alan-agius4
alan-agius4 deleted the pr-takeover-34029 branch September 8, 2026 12:35
@alan-agius4
alan-agius4 restored the pr-takeover-34029 branch September 8, 2026 15:11
@alan-agius4 alan-agius4 reopened this Sep 8, 2026
@angular-robot angular-robot Bot added the area: build & ci Related the build and CI infrastructure of the project label Sep 8, 2026
@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 Sep 8, 2026
@alan-agius4
alan-agius4 merged commit a843fed into angular:main Sep 8, 2026
81 checks passed
@alan-agius4

Copy link
Copy Markdown
Collaborator Author

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

@alan-agius4
alan-agius4 deleted the pr-takeover-34029 branch September 8, 2026 15:27
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: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants