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

mdlr update -s should be -n like in mdlr update #15

Open
BarbzYHOOL opened this issue Apr 26, 2019 · 5 comments
Open

mdlr update -s should be -n like in mdlr update #15

BarbzYHOOL opened this issue Apr 26, 2019 · 5 comments

Comments

@BarbzYHOOL
Copy link
Contributor

To keep consistence with other flags, easier to remember etc

@BarbzYHOOL
Copy link
Contributor Author

seriously this is so ANNOYING

@BarbzYHOOL BarbzYHOOL changed the title mdlr update -s should be mdlr update -n mdlr update -s should be -n like in mdlr update Nov 4, 2019
@BarbzYHOOL
Copy link
Contributor Author

@svarlamov can you tell me where it is in the code to change the flag names ? I can't find it

@svarlamov
Copy link
Member

Not sure exactly what you're looking for as far as functionality, but here's where the flags are in the code: https://github.com/exlinc/mdlr/blob/master/main.go#L31

@BarbzYHOOL
Copy link
Contributor Author

BarbzYHOOL commented Nov 5, 2019

@svarlamov cool thank you but i just realize I can't test it since I don't want to install golang just for changing 2 lines :(

So what i'm saying is that, in different commands like:

mdlr import
mdlr update
mdlr remove
mdlr status

it's either -n or -s to select a specific module.

It should be the same for EVERY COMMAND. Could also be -m so it's easy to remember (m = module) or just n for name

@BarbzYHOOL
Copy link
Contributor Author

seriously, you don't find this crazy?

mdlr remove -n modulename
mdlr import -s modulename
mdlr import -s modulename

??

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

No branches or pull requests

2 participants