-
-
Notifications
You must be signed in to change notification settings - Fork 981
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: update cmd should work normal when set empty string version #4243
fix: update cmd should work normal when set empty string version #4243
Conversation
4b5b66f
to
b5f6abe
Compare
feel free to review it~ @zkochan |
}, | ||
devDependencies: { | ||
'pnpm-foo': '', | ||
'peer-c': '', |
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 didn't know this was possible. What does it mean? Is it the same as *
? Does it work with Yarn/npm?
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.
works fine in yarn, this was a bug here, it will update the devPendencies
(which pkg's version was a empty string ''
) to dependencies
, it's a wrong behaviour before this pr fix...
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 works fine in yarn
/// <reference path="../../../typings/index.d.ts" /> | ||
import './getSpecFromPackageManifest.test' | ||
import './updateProjectManifestObject.test' |
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 whole file is not needed. It was needed when we used tape.
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.
done~
Please also create a changeset for pnpm, with a link to the closed issue. So that the release will create a nice release page like this: https://github.com/pnpm/pnpm/releases/tag/v6.26.1 |
done~ |
cc96c34
to
7daf24c
Compare
closes: #4196