kneasle / wheatley

An AI for Ringing Room that can ring any number of bells to increase the scope of practices.
https://pypi.org/project/wheatley/
MIT License
15 stars 13 forks source link

Add spacing in `--help` for Grandsire calls #211

Closed kneasle closed 2 years ago

kneasle commented 2 years ago

Spacing is required because Python's arg parsing library treats "-1:3.123" in e.g. wheatley 123456789 --single "-1:3.123" as a set of short arguments (same as -.:123). However, inserting whitespace like in "-1: 3.123" means that argparse can't treat it as a set of short options.

An open question is whether it's worth actually explaining the problem in --help but I think the help is long enough already and in most cases this is simply unnecessary detail. It should probably be documented somewhere other than a commit message, though, but I'm not sure of the best place. @centreboard, thoughts?