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

Change option parser for cli #135

Closed
jankowiakmaria opened this issue Oct 29, 2015 · 5 comments · Fixed by #399
Closed

Change option parser for cli #135

jankowiakmaria opened this issue Oct 29, 2015 · 5 comments · Fixed by #399

Comments

@jankowiakmaria
Copy link
Collaborator

as the one currently used (https://github.com/harthur/nomnom):

I can work on it - any recommendations for parser?
https://github.com/tj/commander.js?

@matteofigus
Copy link
Member

Yes I like commander, it would be cool if you could pick up this @jankowiakmaria

@jankowiakmaria
Copy link
Collaborator Author

👍

@nickbalestra
Copy link
Contributor

@jankowiakmaria @matteofigus I used minimist for the cycle.js CLI and I like it (is about 1/5th of the size of commander, and does the job.)

@jankowiakmaria
Copy link
Collaborator Author

Thanks @nickbalestra
I'll try to work on it during the next 20% time.

@jankowiakmaria
Copy link
Collaborator Author

I was able to start working on that ("cli" branch) - I actually decided to use yargs - I didn't like commander for some reason.

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

Successfully merging a pull request may close this issue.

3 participants