Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Open
Milestone
Description
Summary
The strings'None'
vs'none'
are not handled consistently:
- At least MarkerStyle only accepts
'None'
(xrefCleanup 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 donot 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.