-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Refactor the tiny-remapper CLI #6
Conversation
- Version and help is already provided - Parameters are required by default
Also made the `CommandSpec` private
85a6eae
to
adafc6d
Compare
@Earthcomputer let's have you take a look at this since you seem to be the asm guy here |
I didn't change anything there except a commit message. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of unnecessary changes, other than that it looks fine presuming we're happy using this library. I'm not familiar with this library so I'm presuming the annotations are implemented correctly. Just a heads up, JOptSimple is the one that Minecraft uses so may be familiar to more people here.
2c54a5e
to
eb30c85
Compare
Overview
This switches tiny-remapper over to PicoCLI, a lightweight CLI framework. This makes it easier to add new parameters/options, generate help/version info/docs, and give descriptive error messages. It also closes #4, because of the new help command.
Screenshots
Help command
Version command
Error messages:
Manpage