Skip to content
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

bpo-45514: Deprecate importlib resources legacy functions. #29036

Merged
merged 5 commits into from
Nov 24, 2021

Conversation

jaraco
Copy link
Member

@jaraco jaraco commented Oct 18, 2021

  • bpo-45514: Apply changes from importlib_resources@a3ef4128c6
  • Add blurb

Still to do:

https://bugs.python.org/issue45514

@jaraco jaraco requested a review from rhettinger as a code owner October 18, 2021 18:26
@jaraco jaraco removed the request for review from rhettinger October 18, 2021 18:27
@jaraco jaraco marked this pull request as draft October 18, 2021 18:27
@jaraco jaraco marked this pull request as ready for review October 18, 2021 19:23
@jaraco
Copy link
Member Author

jaraco commented Oct 18, 2021

While this PR is ready for review, I want to wait a few days for importlib_resources 5.3 to be in the wild to ensure there aren't any major emergent concerns.

Co-authored-by: Filipe Laíns <lains@riseup.net>
@jaraco jaraco requested review from warsaw and brettcannon October 31, 2021 14:04
@jaraco
Copy link
Member Author

jaraco commented Oct 31, 2021

I've requested a review of this change with Barry and Brett, both of whom have had substantial influence on this interface and both of whom are on the SC. In this discussion, I've made the case that the deprecation is appropriate, but others have made the case that the deprecation happens too quickly, mainly because there won't be one interface that one could depend on across Python 3.13 and 3.8 without adopting a backport.

If either agree the deprecation is too aggressive, I'll back it out to a PendingDeprecationWarning and let it linger indefinitely.

@jaraco
Copy link
Member Author

jaraco commented Oct 31, 2021

While this PR is ready for review, I want to wait a few days for importlib_resources 5.3 to be in the wild to ensure there aren't any major emergent concerns.

There have been no reported issues after issuing the deprecation warning in importlib_resources 5.3.

@jaraco
Copy link
Member Author

jaraco commented Nov 14, 2021

I'd like for this change to be present in early releases of Python 3.11 to limit any surprise and get feedback early. I'd prefer to get a review of this change by next week. Please let me know if you would like more time to consider. If I don't hear anything by 2021-11-20 AOE, I'll plan to merge, but without prejudice; the change can be rolled back or dialed back as needed.

@brettcannon
Copy link
Member

I don't have time to really weigh in on this, so consider me neutral.

@brettcannon brettcannon removed their request for review November 16, 2021 19:38
@jaraco
Copy link
Member Author

jaraco commented Nov 24, 2021

I've decided to move forward with this change, with my rationale here.

@jaraco jaraco removed the request for review from warsaw November 24, 2021 07:49
@jaraco jaraco merged commit d5cd2ef into main Nov 24, 2021
@jaraco jaraco deleted the bpo-45514/deprecate-importlib-resources-legacy branch November 24, 2021 07:51
remykarem pushed a commit to remykarem/cpython that referenced this pull request Dec 7, 2021
…29036)

* bpo-45514: Apply changes from importlib_resources@a3ef4128c6

* Mark legacy functions as deprecated in the docs and link to the migration docs in importlib_resources docs.

* Apply changes from importlib_resources@329ae9d5f2c.

* Indicate importlib.resources as a module.

Co-authored-by: Filipe Laíns <lains@riseup.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants