fix: move pnpm.overrides from package.json to pnpm-workspace.yaml#38724
Merged
Conversation
Co-authored-by: jeremymeng <7583839+jeremymeng@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Update package.json and pnpm-workspace.yaml for pnpm changes
fix: move pnpm.overrides from package.json to pnpm-workspace.yaml
May 28, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR moves the monorepo pnpm override configuration from the deprecated package.json pnpm field into pnpm-workspace.yaml, aligning the repo with pnpm’s current workspace settings behavior and resolving the install warning from issue #38720.
Changes:
- Removed the root
pnpm.overridesconfiguration frompackage.json. - Added the same
@azure/core-rest-pipelinelink override as a top-leveloverridesentry inpnpm-workspace.yaml.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
package.json |
Removes the deprecated pnpm configuration block from the root package manifest. |
pnpm-workspace.yaml |
Adds the pnpm override in the supported workspace configuration location. |
xirzec
approved these changes
Jun 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
pnpm no longer reads the
pnpmfield frompackage.json. Theoverrideskey was silently ignored, producing a[WARN]on everypnpm install.Changes
package.json: removed thepnpm.overridesfield (and the now-emptypnpmobject)pnpm-workspace.yaml: added top-leveloverridessection with the@azure/core-rest-pipelineworkspace linkPackages impacted by this PR
Monorepo root (
@azure/monorepo)Issues associated with this PR
#38720