PEP 615 (zoneinfo) implementation ready for review
Hey all, The feature freeze is coming up on us fast, and the PEP 615 implementation is more or less ready to be integrated into the standard library (may need one or two little tweaks, but it's well past the "minimum viable product" stage). Normally I'd wait longer for someone to volunteer for the task of reviewing, but given the somewhat tight timeline and the fact that the code and tests alone (not including the documentation) are 6000 lines, I figured it's better to give people a head's up that I'm looking for reviewers. I've already had a few reviews on this code when it was first merged to the reference implementation, but there's also a decent chunk of otherwise unreviewed code. - The implementation and tests are in PR #19909: https://github.com/python/cpython/pull/19909 - The documentation and What's New entry is in #20006: https://github.com/python/cpython/pull/20006 There's also one other feature that I did not originally include in the PEP but which I think is a reasonable feature request that we are likely to get, which is a way to list all the time zones available on the system. The reference implementation includes an implementation for that feature in the property test suite, and it would be easy for me to port it over; I'll do that if there are no objections before the feature freeze, I've opened this BPO issue to track the discussion: https://bugs.python.org/issue40536 Thanks! Paul
participants (1)
-
Paul Ganssle