Open
Description
Summary
The strings 'None'
vs 'none'
are not handled consistently:
- At least MarkerStyle only accepts
'None'
(xref Cleanup code for format processing #19291 (comment)). - Maybe other places too. - However, in many places we accept both
'None'
,'none'
(and sometimes also other capitalizations). - Counting occurences, we have 777 'none' and 139
'None'
in our code base.
Proposed fix
Aim at making the API more consistent by:
- Choose one version and use it whenever possible throughout code, docs and examples.
- I propose to choose
'none'
because most other named strings are lowercase. And it's a step further away fromNone
, which may help making it more clear that those two are different.
- I propose to choose
- Make all other places accept that version too.
- For now do not deprecate other writings or conversions. If a way of writing is working now, there'll be lots of code out there using it. It's not worth breaking that.
Note @brunobeltran You might stumble over these inconsistencies when formalizing types.