Enhancement: relax os.path.common input requirements
Offen
Dieses Issue hat noch niemand übernommen.
stdlib
type-feature
- Vorherrschende Sprache
- Python
- Sterne
- 77.2k
- Forks
- 36k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
Feature or enhancement
os.path.commonpath is needlessly strict requiring its paths argument to be a Sequence. The underlying algorithm does not require it and the restriction is an implementation artifact.
Pitch
- Faster code: current implementation goes over its input while, in principle, one iteration is sufficient
- Relaxed argument requirement:
Sequenceis replaced withIterable
Previous discussion
https://discuss.python.org/t/os-path-commonpath-doesnt-need-a-sequence/21543/13
Linked PRs
- gh-99964
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 am Einstiegspunkt os.path.commonpath und sieh dir die bestehende Implementierung und die Tests für dessen Argument paths an. Vergleiche das Verhalten mit der verlinkten Diskussion und überprüfe anschließend, dass iterable Eingaben akzeptiert werden, die Verarbeitung in einem Durchlauf erfolgt und das bestehende Pfadverhalten erhalten bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- operating-systems
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 25/100