Enhancement: relax os.path.common input requirements
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 77.2k
- Forks
- 36k
- Métriques de merge des PR
- Métriques de PR en attente
Description
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
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par le point d’entrée os.path.commonpath et examinez l’implémentation existante ainsi que les tests de son argument paths. Comparez le comportement avec la discussion liée, puis vérifiez que les entrées itérables sont acceptées, que le traitement est effectué en un seul passage et que le comportement existant des chemins reste inchangé.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- operating-systems
- Type d'issue
- Fonctionnalité
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 25/100