Uh oh!
There was an error while loading. Please reload this page.
gh-98040: Remove find_loader, find_module and other deprecated APIs - #98059
Merged
Conversation
warsaw
commented
Oct 7, 2022
MemberAuthor
Still lots of test failures and probably more things to remove, but it's a start! |
warsaw
marked this pull request as ready for review
October 7, 2022 23:14
warsaw
commented
Oct 7, 2022
MemberAuthor
@brettcannon@ericsnowcurrently Well, this branch is already getting pretty big. Pull a thread, unravel a sweater. There's still some things to do (see the ticket), but it's not too late to provide feedback. I was pretty aggressive in removing things that seemed to be obsolete or testing obsolete APIs, so hopefully I wasn't too eager to remove things. Feedback welcome! |
However! pip 22.2.2 still relies on pkgutil.ImpImporter so these things cannot be removed until this issue is resolved: pypa/pip#11501
brettcannon
commented
Oct 14, 2022
Member
Should this PR be a draft until the tests are passing? |
warsaw
marked this pull request as draft
October 15, 2022 19:28
warsaw
commented
Oct 15, 2022
MemberAuthor
Yes, done. The blocker is being worked on upstream so I think pip 23.0 will likely unblock this PR. In the meantime, I'll keep the branch clean as is. |
AA-Turner
left a comment
Member
There was a problem hiding this comment.
There's quite a few places in the docs where we need to update cross references as of this PR (mainly pre-fixing with an !) -- though as they're on lines too far from the changes I can't do inline suggestions.
Would you be happy for me to push to this branch with said docs cross-referencing changes?
A
Uh oh!
There was an error while loading. Please reload this page.
warsaw
commented
May 1, 2023
MemberAuthor
Yes, this would be very helpful, thanks. |
AA-Turner
commented
May 2, 2023
Member
Done, thank you! A |
warsaw
commented
May 2, 2023
MemberAuthor
Thank you! |
Yhg1s
approved these changes
May 3, 2023
jaraco pushed a commit
to python/importlib_metadata
that referenced
this pull request
Jun 7, 2023
Removed ``fixtures.NullFinder``. --------- Co-authored-by: Oleg Iarygin <oleg@arhadthedev.net> Co-authored-by: Adam Turner <9087854+aa-turner@users.noreply.github.com>
jaraco pushed a commit
to python/importlib_metadata
that referenced
this pull request
Jun 8, 2023
Removed ``fixtures.NullFinder``. --------- Co-authored-by: Oleg Iarygin <oleg@arhadthedev.net> Co-authored-by: Adam Turner <9087854+aa-turner@users.noreply.github.com>
clrpackages pushed a commit
to clearlinux-pkgs/pypi-importlib_metadata
that referenced
this pull request
Jun 20, 2023
…6.0 to version 6.7.0 Barry Warsaw (1): gh-98040: Backport python/cpython#98059 Jason R. Coombs (21): Remove unnecessary and incorrect copyright notice. Fixesjaraco/skeleton#78. Replace flake8 with ruff. Fixesjaraco/skeleton#79 and sheds debt. 👹 Feed the hobgoblins (delint). Make substitution fields more prominent and distinct from true 'skeleton' references. (#71) Suppress EncodingWarning in build.env. Ref pypa/build#615. Remove reference to EncodingWarning as it doesn't exist on some Pythons. Inline the NullFinder behavior. Add docstring to test_integration to give some context. Remove Python 2 compatibility in _compat.NullFinder. Move test_interleaved_discovery from test_integration to test_main. Remove test_search_dist_dirs as it was never used. Ref python/importlib_metadata#111. Extract _topmost and _get_toplevel_name functions. Extract _topmost and _get_toplevel_name functions. Capture that _get_toplevel_name can return None. Streamline the test to check one expectation (the standard dist-info expectation is handled by other tests above. Inline the symlink setup. Consolidate PackageDistributions tests. Update _path to jaraco.path 3.6 with symlink support. Utilize the new Symlink in preparing the test case. Update changelog. Remove '__init__.py', not needed. Johan Herland (2): Add test to demonstrate issue with symlinked packages Attempt to fix issue with symlinked packages
fumoboy007 added a commit
to fumoboy007/intellij-community
that referenced
this pull request
Sep 16, 2024
The `MyFinder` subclass of [`MetaPathFinder`](https://docs.python.org/3.11/library/importlib.html#importlib.abc.MetaPathFinder) currently only implements the deprecated `find_module` method. That deprecated method was [removed](python/cpython#98059) in Python 3.12. This change adds an implementation for its replacement method called `find_spec`.
fumoboy007 added a commit
to fumoboy007/intellij-community
that referenced
this pull request
Sep 16, 2024
The included `six.py` is from version 1.9.0. This version is not compatible with Python 3.12, which [removed](python/cpython#98059) the deprecated `MetaPathFinder.find_module` method. This change updates `six.py` to version [1.16.0](https://github.com/benjaminp/six/blob/65486e4383f9f411da95937451205d3c7b61b9e1/six.py), which uses the replacement `MetaPathFinder.find_spec` method.
intellij-monorepo-bot pushed a commit
to JetBrains/intellij-community
that referenced
this pull request
Jul 10, 2025
The `MyFinder` subclass of [`MetaPathFinder`](https://docs.python.org/3.11/library/importlib.html#importlib.abc.MetaPathFinder) currently only implements the deprecated `find_module` method. That deprecated method was [removed](python/cpython#98059) in Python 3.12. This change adds an implementation for its replacement method called `find_spec`. closes#2847 GitOrigin-RevId: b7ebb9f36ac87389298df14f0672ec037da004ef
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.