-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(core): fix invalid yarn parser dependencies grouping key #15647
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Closes: #15603 |
@@ -227,7 +227,7 @@ function groupDependencies( | |||
const resolutionMap = new Map<string, YarnDependency>(); | |||
const snapshotMap = new Map<YarnDependency, Set<string>>(); | |||
Object.entries(dependencies).forEach(([key, snapshot]) => { | |||
const resolutionKey = `${snapshot.resolution}${snapshot.integrity}`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was a typo - grouping is only created for Yarn Classic, which has resolved
, not for Yarn Berry which has a resolution
field
(cherry picked from commit 3f4ba6d)
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
The yarn parser uses an invalid key for grouping dependencies causing different snapshots to be grouped together.
Reported by @yharaskrik on Slack: https://nrwlcommunity.slack.com/archives/CMFKWPU6Q/p1677717333068159
Current Behavior
Expected Behavior
Related Issue(s)
Related to #15364
Fixes #15603