paulk-asert commented on pull request #1467: URL: https://github.com/apache/groovy/pull/1467#issuecomment-765401475
This change might be okay but possibly not needed. This currently works: ``` def cli = new CliBuilder() cli.a(args: 1, longOpt: 'abc', type: double, 'abc') def options = cli.parse(['--abc', '0.0']) assert options.abc instanceof Double ``` CliBuilder doesn't use the type info capability offered by commons cli but instead performs its own conversion: https://github.com/apache/groovy/blob/master/subprojects/groovy-cli-commons/src/main/groovy/groovy/cli/commons/OptionAccessor.groovy#L118 ---------------------------------------------------------------- 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: [email protected]
