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.

Sign up for GitHub

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

Jump to bottom

Support for short flags #49

Closed
qustavo opened this issue Sep 28, 2017 · 4 comments
Closed

Support for short flags #49

qustavo opened this issue Sep 28, 2017 · 4 comments
Labels
Hacktoberfest In Progress

Comments

@qustavo
Copy link
Owner

qustavo commented Sep 28, 2017

Right now we just support long flag format (ie -port, -config, etc). It would be nice to provide the short version for: port, config and version.

https://github.com/alecthomas/kingpin looks like a great candidate for this, but any other should be fine.
@dnguy078
Copy link
Contributor

dnguy078 commented Sep 28, 2017 •
edited
Loading

Hi @gchaincl , could you please review this #50

@dnguy078 dnguy078 mentioned this issue Sep 28, 2017
Merged
@sponomarev sponomarev mentioned this issue Sep 28, 2017
Closed
@sponomarev
Copy link
Contributor

Hey, @gchaincl!

#51 Here is my solution 😄

@qustavo
Copy link
Owner Author

qustavo commented Sep 29, 2017

Cool, both looks really well, as I don't have a strong opinion on which library to use pflag, o kingpin, I'll add comment to both, the first one in resolve them, gets merged!

@qustavo
Copy link
Owner Author

qustavo commented Oct 2, 2017

Closed by #50

@qustavo qustavo closed this as completed Oct 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Hacktoberfest In Progress
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@qustavo @dnguy078 @sponomarev