-
Notifications
You must be signed in to change notification settings - Fork 47
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
shorter bin install commands #170
Conversation
hey! thx for the contribution! Your PR gave me an idea. How about if we add a new command like We could include the shorthands that you're proposing here as a default during config init or maybe through Thoughts? |
I think for the moment smart choose is more important. I mean for this case, 3 should be chosen. I noticed there are issues out on the list.
|
I agree... but I think moving that file shorthands to the
yes, agree. I have some ideas on how to improve this, just need to find the time to try them out. |
it helps ... |
would you be interested in making this refactor? I can tackle whenever I find some bandwidth though. |
I think from the perspective of keeping it simple it's not necessary, with this PR, the most widely used tools should be covered, and the else could be installed by |
👍. I'll refactor when I find the time. In the meantime I'll hold this PR since I don't want to hardcode custom shorthands in the code that will require a new release every time someone wants to add a new one. |
Can you enable the discussions feature for this repo? |
Sure, done |
@marcosnils thanks, #172 |
closing since this didn't get too much traction and it has been stale for a while |
close #171