#288 Remove space between arg name and value, support of arg repeating with each value#310
#288 Remove space between arg name and value, support of arg repeating with each value#310bugy merged 3 commits intobugy:masterbugy/script-server:masterfrom miksir:space_after_arg_and_multiple_argmiksir/script-server:space_after_arg_and_multiple_argCopy head branch name to clipboard
Conversation
|
Hi @miksir , thanks for the PR, the changes look good to me, but I have some concerns regarding naming. Regarding repeat_arg: at the moment for the -p (or --flag, or ---arg=) i use config called param. I think for consistency it would be better to keep it. What do you think? |
|
I am really dont care about naming :) Just need time for search and replace :) Will try to rename tomorrow. |
|
So any suggestions how to call param_space? :) param_value_split? :) repeat_arg to repeat_param? or repeat_param_with_each_value? Renaming param to option require renaming of |
|
|
|
Or may be be same_arg_param |
+ Option for add arg name with each value of multioption type
d5ddb9b to
280a5ac
Compare
|
Renamed |
|
There are 2 web tests failing, would you mind having a look? Or should I? |
|
Oh, I will check. |
|
Probably fired because I changed visibility of some fields when they are unused (they was disabled before) |
Issue #288