Closed remkop closed 4 years ago
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 notifications@github.com 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 https://github.com/petermr/ami3/issues/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.
Done:
setUnmatchedOptionsAllowedAsOptionParameters(false)
(Your https://github.com/remkop/picocli/issues/639 feature request was finally implemented in 4.4)picocli-codegen
annotation processor in pom.xml
, to get better validation: this gives some compile-time errors and warnings for invalid annotations.
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.