-
-
Notifications
You must be signed in to change notification settings - Fork 32.6k
gh-118974: Add decorator
argument to make_dataclass
#122723
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
13e940a
to
431addd
Compare
Misc/NEWS.d/next/Library/2024-08-06-07-24-00.gh-issue-118974.qamsCQ.rst
Outdated
Show resolved
Hide resolved
I'm not crazy about the name |
Other suggestions at the core sprint are |
I think But anyway, I'm happy to make the change to use |
@larryhastings : Any thoughts on the parameter name? I'm leaning towards just |
I think
That strikes me as an excellent fit. Also, I prefer I think I'm the dude who suggested |
Convinced! Just pushed a commit fixing this. |
Thanks, @larryhastings for the excellent analysis (I expected nothing less!). I'll review the patch when I have some free time. |
dataclass_factory
argument to make_dataclass
decorator
argument to make_dataclass
decorator
argument todataclasses.make_dataclass
for custom dataclass transformation support #118974