Skip to content

fix: peer dependency rules should not change the lockfile#7758

Merged
zkochan merged 3 commits intomainfrom
fix-peer-warns
Mar 13, 2024
Merged

fix: peer dependency rules should not change the lockfile#7758
zkochan merged 3 commits intomainfrom
fix-peer-warns

Conversation

@zkochan
Copy link
Member

@zkochan zkochan commented Mar 13, 2024

No description provided.

@zkochan zkochan merged commit aa33269 into main Mar 13, 2024
@zkochan zkochan deleted the fix-peer-warns branch March 13, 2024 16:55
@47hao
Copy link

47hao commented Dec 4, 2024

The changes in this PR introduced a bug in peer dependency error reporting.

When the effects of peerDependencyRules were switched from patching to reporting-only, they moved from upstream of the PeerDependencyIssuesError throw to downstream of it.

As a result, in the following conditions:

  • strict-peer-dependencies is enabled
  • There are peer dependency issues
  • The issues are fixed via peerDependencyRules

A peer dependency error is thrown. Nothing is reported since the render function's logic realizes that all issues are fixed. The observed result is pnpm install silently fails with exit code 1 and no errors are logged.

Here's a repro, and more detailed info.

@zkochan
Copy link
Member Author

zkochan commented Dec 5, 2024

Please file an issue with details then. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants