-
Notifications
You must be signed in to change notification settings - Fork 29
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
chore(deps): upgrade dependencies #267
Conversation
upgrading dependencies to update minimist resolutions to use 1.2.5
@@ -2,144 +2,152 @@ | |||
# yarn lockfile v1 | |||
|
|||
|
|||
"@babel/code-frame@^7.0.0", "@babel/code-frame@^7.5.5", "@babel/code-frame@^7.8.3": |
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.
Did you manually update yarn.lock to cover this? I thought that with yarn, the preferred pattern was to update package.json
with a resolutions
property to enable yarn to handle this itself; was that not an option here?
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 did not manually update yarn. I just deleted yarn.lock & did yarn install again to update all packages. That fixed the issue. Alternatively i could have also added resolution in package.json on npm to fix this issue. But, chose this option since we anyway try to be up to date with dependabot.
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.
Question inline
🎉 This PR is included in version 2.3.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description of changes
upgrading dependencies to update minimist resolutions to use 1.2.5 (upgrading npm from 6.13.6 to 6.14.4 fixes this issue, but i just upgraded all packages instead.)
1.2.0 version was used under node_modules/npm/node_modules/rc/node_modules/minimist/package.json
& 0.0.8 under /home/vsts/work/1/s/node_modules/npm/node_modules/minimist/package.json
Pull request checklist
fix:
,feat:
, etc, and is suitable for user-facing release notes)BREAKING CHANGE:
yarn precheckin