-
Notifications
You must be signed in to change notification settings - Fork 5
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
Bump picocli to 4.4.0 #49
Labels
dependencies
Pull requests that update a dependency file
Comments
Thanks very much Remko.
picocli is central to how we work.
At present I don't recommend abbreviations to the openVirus group as the
full names help us understand the science. But they are valuable for power
users.
P.
…On Sun, Jul 5, 2020 at 9:24 AM Remko Popma ***@***.***> wrote:
This version has some parser improvements that allow applications to be
more strict and reject unknown options as option values ( a feature request
@petermr <https://github.com/petermr> made about a year ago finally
landed).
Also has support for abbreviated commands and/or options if we want to use
this in AMI.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#49>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFTCS2DESFZDD5OHXKFHHLR2A2CBANCNFSM4OQYQ7YQ>
.
--
Peter Murray-Rust
Founder ContentMine.org
and
Reader Emeritus in Molecular Informatics
Dept. Of Chemistry, University of Cambridge, CB2 1EW, UK
|
Makes sense. Abbreviations can also introduce ambiguity that didn’t previously exist. In our case probably better to keep that feature switched off. |
remkop
added a commit
that referenced
this issue
Jul 6, 2020
Done:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This version has some parser improvements that allow applications to be more strict and reject unknown options as option values ( a feature request @petermr made about a year ago finally landed).
Also has support for abbreviated commands and/or options if we want to use this in AMI.
The text was updated successfully, but these errors were encountered: