Closed yusuke-noda closed 10 months ago
Thanks for doing this @yusuke-noda!
Relatedly, I released a change with 9.5.1 so that in a config file you can provide a two-item array directly instead of a single string to avoid the ambiguity of the splitting. But of course we'll still always need the splitting logic for the CLI.
I'll fully review and merge this shortly. Before releasing I might also add some deprecation warnings, because what I'd like to do long term is enforce quoting either side of the string if it contains colons, so we don't have to do such clever splitting with awareness of all the platform quirks.
Hi @yusuke-noda,
Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! πΎ
In return for this generous offer we hope you will:
On behalf of the Cucumber core team, Aslak HellesΓΈy Creator of Cucumber
Referring to #2178
π€ What's changed?
Reimplement OptionSplitter and fix the issue discussed in #2178 .
β‘οΈ What's your motivation?
Fixes #2178
π·οΈ What kind of change is this?
β»οΈ Anything particular you want feedback on?
π Checklist:
This text was originally generated from a template, then edited by hand. You can modify the template here.