-
Notifications
You must be signed in to change notification settings - Fork 254
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
feat: allow to select from registry list if name is undefined #191
Conversation
🦋 Changeset detectedLatest commit: 84f7e95 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
71573bf
to
868f74f
Compare
Thanks @iosh. The test ci worked well. |
OK, I will add test to this PR (in this weekend) |
Thanks again. 👍🏻 |
645c1c0
to
df01cb8
Compare
df01cb8
to
3bb88bd
Compare
I had merged the pr into this branch. Thanks! But the test ci failed. @iosh I think this module can be removed. If you think so, please fix it by this branch. |
7d56047
to
84f7e95
Compare
}); | ||
|
||
expect(message).toBe(`? Please select the registry you want to use (Use arrow keys) | ||
${isWin ? '>' : '❯'} 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.
@iosh It is used to adapt to mac and windows. Otherwise, test ci will fail....
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.
Thanks again @chouchouji
Issue
close #187
Result
use.mp4