Fix format of dataclasses' unsafe_hash default value
#116532
Conversation
|
The intent is that it's saying any value that evaluates as false in a boolean context. It's a little confusing to say "If false (the default is |
|
The goal here was just to make the formatting consistent with other parameters. For instance for
So this switches from: ``False``to |
| - *unsafe_hash*: If false (the default), a :meth:`~object.__hash__` method | ||
| is generated according to how *eq* and *frozen* are set. |
There was a problem hiding this comment.
Perhaps "If true, force dataclasses to create a __hash__ method, even when it may not be safe to do so. Otherwise, generate a __hash__ method according to how eq and frozen are set. The default value is False."
or words to that effect?
There was a problem hiding this comment.
Yes, I think this is the best approach. It could be extended to the default=True values as well: put the default at the end.
To be in line with the other parameters. I wonder what is the precedent here: should every default value be wrapped in preformatted text? For the other dataclass parameters, only the default value is, but not the actual if true/false:
📚 Documentation preview 📚: https://cpython-previews--116532.org.readthedocs.build/