You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For the _allow_empty keyword I initially added the underscore to make a name conflict with a placeholder less likely. This was exaggerated to begin with but after #99 we can change it with confidence. But what should we do exactly?
leave it be
no work, but I am no fan of the underscore
rename to allow_empty
definitely a better name, but maybe a bit overkill
rename to on_empty with the options "raise" | "warn" | "???"
would align with on_parse_error & add an option ("warn")
but what name should the third option be? ("ignore" seems wrong)
For the
_allow_empty
keyword I initially added the underscore to make a name conflict with a placeholder less likely. This was exaggerated to begin with but after #99 we can change it with confidence. But what should we do exactly?allow_empty
on_empty
with the options"raise" | "warn" | "???"
on_parse_error
& add an option ("warn")"ignore"
seems wrong)@veni-vidi-vici-dormivi I am asking once again for your opinion 🤷♂️
The text was updated successfully, but these errors were encountered: