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(npm): package-lock.json v3 root project out of sync with package.json #23495
Conversation
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 must be a new behavior, because we never had issues before. which npm version ate you using? does this argument works with older npm version?
… more proper lockfile maintaince
It bothers me a little that no unit tests are affected by this change, though 😄 012c0ac I think the bug was introduced here, but I'm not 100% https://github.com/firecow/renovate-reproduce-yarn/pulls Here is a yarn example, which shows that no bug is present in renovate's yarn lock file PR's |
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.
I wouldn't call npm lock file version 3 new.
But yeah, kindda new 👍
Edit: This was a reply to your comment @viceice
failed tests and linting |
I just ran them locally, and they worked if I set lockfile version condition to be explicit for v3. Can you run them again? @viceice |
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.
🤔 do we've a version 4?
Would be better to do: is.number && ... |
Of lockfile standard? Seems v3 is the latest. https://docs.npmjs.com/cli/v7/configuring-npm/package-lock-json#lockfileversion |
🎉 This PR is included in version 36.30.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes
Allow "Massage lockfile" for lock file version 3
Context
Ensure that package-lock.json root project is in sync with package.json.
See #23488
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via: