Skip to content
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

Merged
merged 3 commits into from Jan 26, 2022
Merged

Remove use of alias #276

merged 3 commits into from Jan 26, 2022

Conversation

ryandberger
Copy link
Member

@github-actions
Copy link

Jest Unit Test Statistics

0 files  ±0  0 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 821533e. ± Comparison against base commit d28f64f.

@github-actions
Copy link

BigTest Unit Test Statistics

0 files  ±0  0 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 821533e. ± Comparison against base commit d28f64f.

@sonarcloud
Copy link

sonarcloud bot commented Jan 24, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link

@skomorokh skomorokh left a 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?

Copy link
Contributor

@mkuklis mkuklis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice cleanup @ryandberger!

Copy link
Contributor

@MikeTaylor MikeTaylor left a 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
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yikes!!

@ryandberger
Copy link
Member Author

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?

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.

@ryandberger ryandberger merged commit be642f9 into master Jan 26, 2022
@ryandberger ryandberger deleted the STCLI-155 branch January 26, 2022 21:51
Comment on lines +220 to 221
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.

Copy link
Member

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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants