This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author larry
Recipients BTaskaya, grahamd, larry, ncoghlan, ofey404, ping, yselivanov
Date 2022-03-25.22:39:02
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1648247942.84.0.00123826761824.issue46761@roundup.psfhosted.org>
In-reply-to
Content
I heard back from both Samuel Colvin (Pydantic) and Sebastián Ramírez (FastAPI).  They said neither of them use update_wrapper with partial objects.  They also took a peek in a bunch of other projects (FastAPI, Typer, SQLModel, Asyncer, SQLAlchemy, Trio, and AnyIO) and nobody was doing it.

So honestly it seems like nobody (but me!) calls update_wrapper on partial objects, and we can just fix it.

Graham, any final thoughts before we start pulling levers and merging PRs?  For now I just want to fix this bug.  I'm in favor of re-engineering the relevant objects so they write their own __signature__ objects, so inspect.Signature doesn't have to understand the internals of objects from other modules.  But maybe we save that for another day.
History
Date User Action Args
2022-03-25 22:39:02larrysetrecipients: + larry, ping, ncoghlan, grahamd, yselivanov, BTaskaya, ofey404
2022-03-25 22:39:02larrysetmessageid: <1648247942.84.0.00123826761824.issue46761@roundup.psfhosted.org>
2022-03-25 22:39:02larrylinkissue46761 messages
2022-03-25 22:39:02larrycreate