-
-
Notifications
You must be signed in to change notification settings - Fork 750
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 "prev" command in 1.3 #184
Comments
I personally like it to be quite honest 😄 I would like to further suggest implementing Would a PR for this be welcome? |
|
Wasn't aware of that, you're completely right that we shouldn't use I'd leave |
I'd consider that as well. I don't have much use for aliases anyway. There shouldn't be lots of ways to do one thing, especially in such a simple tool. |
👍 |
👍 I never use prev. I would just switch to the version number. It's not like we are typing that much more to achieve the same outcome. |
👍 Same over here |
I am going to submit a PR to remove this. I ran into an interesting case that breaks the state of n when using prev. n 0.12.0 |
I think prev is weird also
|
This would have to constitute a new major version, though. Because semver |
I use |
I just took a look at your project. I have to say it is very impressive. Good work. As for the way you are using |
prev was pulled in 2.0.0. |
what are everyone's thoughts on this? I personally find it more confusing than helpful, and prone to off-by-one situations
The text was updated successfully, but these errors were encountered: