exlinc / mdlr

MDLR ("Modular") gives you declarative and easy Git 'submodules without the pain of submodules'
Apache License 2.0
27 stars 2 forks source link

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

Open BarbzYHOOL opened 5 years ago

BarbzYHOOL commented 5 years ago

To keep consistence with other flags, easier to remember etc

BarbzYHOOL commented 4 years ago

seriously this is so ANNOYING

BarbzYHOOL commented 4 years ago

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

svarlamov commented 4 years ago

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 commented 4 years ago

@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 commented 4 years ago

seriously, you don't find this crazy?

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

??