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
Remove use of alias #276
Remove use of alias #276
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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.
Hooray 🎉 it's the beginning of the end of this, I presume the idea is you're marking it as a deprecated now in anticipation of removing it with the next major release?
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.
Nice cleanup @ryandberger!
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.
Very welcome!
@@ -1,6 +1,6 @@ | |||
# Stripes CLI Commands | |||
|
|||
Version 1.20.0 | |||
Version 2.5.0 |
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.
Yikes!!
Yes, seems appropriate to remove with a major release, though I'd want to determine a path for someone to manually remove global aliases when the command no longer exists. |
Note that in order for local code repositories to be used for a reference made in another module or platform `package.json` file, the local code repository version must fall in the requested version range. Otherwise, `yarn` will fetch from the remote repository instead. | ||
|
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 is an especially helpful note. Thanks for adding it!
alias
command and updated documentation in favor ofworkspace
alias
was deprecated in the description and that an upcoming release ofyargs
should provide support that our documentation generator can leverage. See [Question] How do I mark a command as deprecated? yargs/yargs#2115 for details.