You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by GitBox <gi...@apache.org> on 2020/02/24 15:57:38 UTC
[GitHub] [commons-cli] waso opened a new pull request #33: added separate
setter for Builder.opt
waso opened a new pull request #33: added separate setter for Builder.opt
URL: https://github.com/apache/commons-cli/pull/33
I would like to add (suggest) new setter method to set name of the option ('opt') in Option.Builder class outside of constructor.
So something like this would be allowed:
`Option.builder().opt("a")`
which would be equivalent to:
`Option.builder("a")`
Personally I find it more readable, maybe others will agree as well.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [commons-cli] coveralls commented on issue #33: added separate
setter for Builder.opt
Posted by GitBox <gi...@apache.org>.
coveralls commented on issue #33: added separate setter for Builder.opt
URL: https://github.com/apache/commons-cli/pull/33#issuecomment-590412632
[](https://coveralls.io/builds/28923715)
Coverage increased (+0.006%) to 96.364% when pulling **bcef4c59f8d65e9eccfce6444bf5bc144573e6c1 on waso:feature/builder_opt** into **c12260c05ebcd27558265395df79b8b049534fbd on apache:master**.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services