Add `discover()` method to (all) finders
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 77.2k
- Forks
- 35.9k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
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
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die bestehenden discover()-Implementierungen für PathFinder und FileFinder zu lesen, und untersuche anschließend die übrigen Finder-Typen: BuiltinImporter, FrozenImporter, WindowsRegistryFinder und zipimporter. Ermittle, welche Finder die Methode sinnvoll unterstützen können, und verifiziere, dass sich jede ausgewählte Implementierung konsistent mit den bestehenden verhält. Erledigt ist die Aufgabe, wenn die anwendbaren, noch nicht geprüften Finder implementiert und durch die relevanten Tests des Projekts abgedeckt sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- backend
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 48/100