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

Parse args from CLI with OptionParser.parse #162

Closed
nappex opened this issue Jan 21, 2023 · 2 comments · Fixed by #189
Closed

Parse args from CLI with OptionParser.parse #162

nappex opened this issue Jan 21, 2023 · 2 comments · Fixed by #189
Assignees
@nappex
Labels
cli do it first

Comments

@nappex
Copy link
Collaborator

nappex commented Jan 21, 2023

Currently, we parse CLI args and options just with pattern matching #145. It will be nice if we parse args from CLI by built-in module OptionParse check official docs or tutorial - elixir school. This approach ensure to handle errors more elegant with help message and so on. Currently we get some ugly error. It is not very user friendly interface.
@Glutexo
Copy link
Owner

Glutexo commented Jan 23, 2023

I completely agree!

@Glutexo Glutexo mentioned this issue Jun 2, 2023
Closed
@nappex
Copy link
Collaborator Author

nappex commented Jun 2, 2023

Solve this before issue #182!

@nappex nappex mentioned this issue Jun 9, 2023
Closed
@nappex nappex mentioned this issue Oct 3, 2023
Merged
@nappex nappex added the cli label Oct 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nappex nappex

Labels
cli do it first
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use OptionParser to parse component nappex/onigumo
2 participants
@Glutexo @nappex