| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
Co-authored-by: Thomas Grainger <tagrain@gmail.com>
There was a problem hiding this comment.
Please could you note the deprecations in https://docs.python.org/3.12/library/importlib.html with deprecated-removed?
and list under https://docs.python.org/3.12/whatsnew/3.12.html#pending-removal-in-python-3-13 ?
Sorry, something went wrong.
|
According to PEP 387, if these are being deprecated in 3.12, is 3.14 the earliest they can be removed? |
Sorry, something went wrong.
There was a problem hiding this comment.
Added some questions.
Sorry, something went wrong.
|
When you're done making the requested changes, leave the comment: I have made the requested changes; please review again. |
Sorry, something went wrong.
|
Honestly, I don't think all this is necessary. What would be the harm if they stay importable from both places? |
Sorry, something went wrong.
From a backward compatibility question, none. But we should be crisp in the documentation about where the proper place to import them from is, and add deprecations for the old places. |
Sorry, something went wrong.
Yes, although I contend it would be preferable to deprecate these in 3.11 for removal in 3.13, given that is the prior intention. |
Sorry, something went wrong.
|
Okay! I thought the 3.11 beta period was too late for new deprecations, but let's check with @pablogsal to see if this can be backported to 3.11. |
Sorry, something went wrong.
Indeed, unfortunately is too late to add new deprecations to 3.11, specially given how unstable 3.11 is already. Sadly this needs to be done starting in 3.12 :( |
Sorry, something went wrong.
Co-authored-by: Thomas Grainger <tagrain@gmail.com>
|
I have made the requested changes; please review again. |
Sorry, something went wrong.
|
Thanks for making the requested changes! @warsaw: please review the changes made to this pull request. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Fixes #93963
Automerge-Triggered-By: GH:jaraco