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
Many thanks to authors of cattrs for the minimalistic design and simplicity!
Do you think it is reasonable to add more hooks, covering frequently used classes from stdlib?
In this specific case I mean zoneinfo.ZoneInfo, both structuring and unstructuring.
I probably could come up with PR for this.
The text was updated successfully, but these errors were encountered:
Thanks for the offer. The problem with adding these less commonly used types is that they add a tiny bit of overhead to converter startup, and possibly the import time, and this has to be borne by everyone.
What we could do is add a section to the recipes doc with hooks for these stdlib types. I'd merge that in no problem.
Going to close this now in an attempt to keep the issue list manageable. 😇
Many thanks to authors of
cattrs
for the minimalistic design and simplicity!Do you think it is reasonable to add more hooks, covering frequently used classes from stdlib?
In this specific case I mean
zoneinfo.ZoneInfo
, both structuring and unstructuring.I probably could come up with PR for this.
The text was updated successfully, but these errors were encountered: