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

Replace commanderjs with own parser #10

Open
sQVe opened this issue Oct 18, 2018 · 1 comment
Open

Replace commanderjs with own parser #10

sQVe opened this issue Oct 18, 2018 · 1 comment
Labels
priority: medium type: maintenance

Comments

@sQVe
Copy link
Owner

sQVe commented Oct 18, 2018

The currently implementation works around commanderjs instead of working with. We should probably write our own argv parser that is "easy:ish" to test. If needed we can use the basic functionality in yargs.
sQVe added a commit that referenced this issue Dec 12, 2018
@sQVe
fix: safe guard against no / wrong input
7b1f7d9 
This is missing a safe guard against cases where a user inputs a command
that is missing. Currently there is no good way to solve with this
commander.js. We need to implement #10
solve this.
@sQVe
Copy link
Owner Author

sQVe commented Dec 13, 2018

We need to be able to safe guard against inputting a command which does not exist. In those cases we would like to display the help information.

sQVe added a commit that referenced this issue Dec 20, 2018
@sQVe
fix: safe guard against no / wrong input
6ed1fbd 
This is missing a safe guard against cases where a user inputs a command
that is missing. Currently there is no good way to solve with this
commander.js. We need to implement #10
solve this.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority: medium type: maintenance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sQVe