Intake icon indicating copy to clipboard operation
Intake copied to clipboard

Fixed issue with Parametric commands failing due to CommandArgs not being in namespace.

Open me4502 opened this issue 8 years ago • 1 comments

Currently, if you try to setup Parametric commands, it fails due to the DefaultModule expecting CommandArgs to be in the namespace when parser.parseArguments is called. This moves it to where it's defined, meaning it's available.

me4502 avatar Mar 18 '17 12:03 me4502

I'm experiencing this issue too and looked into it before I found this.

Any reason we can't use the CommandArgs parameter passed to CommandArgsProvider directly? It seems to go through OptionType#transform(CommandArgs), but that shouldn't matter since declaring the CommandArgs parameter as @Switch or @Optional makes little to no sense, which means that the CommandArgs passed as parameter should generally be the same as these in the Namespace.

literalplus avatar Apr 29 '17 20:04 literalplus