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
Support __doc__ argument in create_model() #7863
Support __doc__ argument in create_model() #7863
Conversation
please review |
Hi @chris-spann, Thanks so much for your contribution! ⭐ I think we'll want to add support explicitly for the Please update 👍 |
Thank you @sydney-runkle for the review! Please let me know if I've adequately addressed your comment in my latest commit. Please review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @chris-spann,
LGTM other than a few small changes I left comments for. Once you add those small changes, I'll approve and merge.
Thanks again for your contribution. This looks great 🌟
Apologies, you were actually spot on with the work from your second commit. I forgot that we actually don't support docstring inheritance in V2, that was only in V1 (see my comment here). Your changes look great, I'll merge here in a sec 👍. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for the contribution! 🎈
Change Summary
Fixes runtime error when using
__doc__
argument forcreate_model()
Related issue number
fix #7481
Checklist
Selected Reviewer: @samuelcolvin