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

flags improvements #14

Closed
dcaba opened this issue Oct 28, 2017 · 10 comments
Closed

flags improvements #14

dcaba opened this issue Oct 28, 2017 · 10 comments

Comments

@dcaba
Copy link
Collaborator

dcaba commented Oct 28, 2017

I think we can improve how we recover arguments:

As, I think a milestone for this project, is to create debian/rpm packages and a docker image, and allow users to use this tool as any other you may have in your OS, and the default flags in golang are not posix compliant plus they dont support "short letters"...

@dcaba
Copy link
Collaborator Author

dcaba commented Oct 31, 2017

dismissing creating flags in the init function. This requires all vars to become global (not nice)

@dcaba
Copy link
Collaborator Author

dcaba commented Oct 31, 2017

https://github.com/spf13/pflag ; previous reference is no longer maintained (it actually does not work well)

@dcaba dcaba self-assigned this Oct 31, 2017
@chadell
Copy link
Collaborator

chadell commented Nov 1, 2017

I would assume this issue is fully addressed by the Merged PR, isn't it?

@dcaba
Copy link
Collaborator Author

dcaba commented Nov 1, 2017 via email

@dcaba
Copy link
Collaborator Author

dcaba commented Nov 2, 2017

Oggh, I found this one:
https://github.com/alecthomas/kingpin

Quite better! I'd vote for switching to it!! See this:
https://github.com/alecthomas/kingpin#complex-example

Simple but effective ;)

@chadell
Copy link
Collaborator

chadell commented Nov 2, 2017

I like it! it has methods that simplifies a lot the code after parsing.

@dcaba
Copy link
Collaborator Author

dcaba commented Nov 4, 2017

releasing ownership. I'm working in the closure logic (next steps on this become orphan, but i can pick in the future)

@dcaba dcaba removed their assignment Nov 4, 2017
@dcaba
Copy link
Collaborator Author

dcaba commented Nov 4, 2017

another alternative, very common: https://github.com/spf13/cobra

@chadell
Copy link
Collaborator

chadell commented Nov 25, 2017

working in #38

@chadell
Copy link
Collaborator

chadell commented Dec 8, 2017

done by #38

@chadell chadell closed this as completed Dec 8, 2017
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