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

Added ArgumentBuilder#transform #38

Open
wants to merge 47 commits into
base: master
Choose a base branch
from
Open

Conversation

jontejj
Copy link
Contributor

@jontejj jontejj commented May 3, 2018

No description provided.

Jonatan Jönsson and others added 30 commits May 2, 2018 13:48
Will be available in maven central in a couple of hours.
This is a non backwards-compatible change. But given that it's more comforting to have all
commands run successfully than only half, I think this is the better choice after all.

Fixes #39
Fixed meta description in usage for transform'ed arguments.
jontejj and others added 11 commits May 7, 2018 20:30
If they are of the type ArgumentException that is.
Now tab-completions can be used for jargo programs by simply using:
mvn package (in your project, to produce a runnable jar file)

And then the user needs to put this in their ~/.bash_profile file:
alias my-app="java -jar /my-app/target/app.jar"
complete -o default -o bashdefault -o nospace -C my-app "my-app"

Where my-app is the name of your program.

Notes:
* CommandLineParser#noCompleter turns this function off, it's enabled by default.
* VariableArity/Arity/etc arguments gets completions
* help arguments gets completions for what arguments you can get help for
* Lowercase support for enum arguments
* Not limiting suggestions

Other fixes:
* Removed Lines.NEWLINE and StringUtils.UTF8 as replacements exists in jdk 7
* Improved the security test
Also bumped version in readme.
@jontejj
Copy link
Contributor Author

jontejj commented May 28, 2018

@petternordholm du får gärna trycka in denna :) Prova gärna lite tab-completions också :)

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

Successfully merging this pull request may close these issues.

1 participant