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

meteor authorized <site> -a <username> doesn't work (only --add works) #2155

Closed
acemtp opened this Issue May 18, 2014 · 2 comments

Comments

Projects
None yet
3 participants
@acemtp

acemtp commented May 18, 2014

In the meteor authorized help, it says we can do --add or -a:

--add, -a     add an authorized user
--remove, -r  remove an authorized user
--list        list authorized users (the default)

but when we do with a -a, it says -a is an unknown option.

$ meteor authorized webdo -a peernohell
-a: unknown option.

--add works

$ meteor authorized webdo --add peernohell
webdo.meteor.com: added peernohell

@Slava Slava closed this in 6f8e18b May 18, 2014

@Slava

This comment has been minimized.

Member

Slava commented May 18, 2014

Looks like the short aliases were removed in b460fe8 but I can see no reason why - as of 6f8e18b the authorized self-tests seem to pass. Thanks @acemtp for the heads up.

@estark37

This comment has been minimized.

Contributor

estark37 commented May 21, 2014

With some advice from @glasser, we decided to remove the short aliases because our command option parsing makes it so that all flags have to be consistent in whether or not they are boolean. Because we might easily want to have boolean -a and -r flags for other commands (--all-, --recursive, for example), we decided to actually not add them back in for meteor authorized, which is used relatively infrequently. We removed the short aliases from the help text instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment