Add `discover()` method to (all) finders
Open
Nobody has claimed this yet.
stdlib
topic-importlib
type-feature
- Dominant language
- Python
- Stars
- 77.2k
- Forks
- 35.9k
- PR merge metrics
- PR metrics pending
Description
Feature or enhancement
Proposal:
It would be nice to implement discover() for all finders where that can be reasonably achieved.
-
BuiltinImporter -
FrozenImporter -
WindowsRegistryFinder -
PathFinder -
FileFinder -
zipimporter
Has this already been discussed elsewhere?
No response given
Links to previous discussion of this feature:
No response
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the existing discover() implementations for PathFinder and FileFinder, then inspect the remaining finder types: BuiltinImporter, FrozenImporter, WindowsRegistryFinder, and zipimporter. Determine which finders can reasonably support the method and verify that each selected implementation behaves consistently with the existing ones. Done means the applicable unchecked finders are implemented and covered by the project's relevant tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100