python / python/cpython

Consolidate `pathlib.PurePath.match()` and `full_match()`

Offen
#139,282 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

stdlib topic-pathlib
Vorherrschende Sprache
Python
Sterne
77.2k
Forks
35.9k
PR-Merge-Kennzahlen
PR-Kennzahlen ausstehend

Beschreibung

Whichever fool (me!) added pathlib.PurePath.full_match() should have given more consideration to adding an argument to match() instead of a new method. The behaviour isn't that different. The fact we have two very similar methods is a bit of a wart. It's hopefully not too late to rectify.

Linked PRs
  • gh-139311

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Überprüfe die im Issue beschriebenen APIs pathlib.PurePath.match() und full_match() und untersuche anschließend den verlinkten PR gh-139311, um zu sehen, wohin sich die Arbeit verschoben hat. Als abgeschlossen gilt die Erreichung einer vereinbarten Zusammenführung der beiden ähnlichen Methoden, ohne die aktuelle API-Inkonsistenz ungelöst zu lassen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
api
Issue-Typ
Refactoring
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
15/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.