`peel` arguments inconsistent.
Ouverte
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 1.7k
- Forks
- 408
- Merge moyen
- 2 j 57 min
- PR mergées (30 j)
- 7
Description
peel is implemented twice, once in object.c and once in reference.c. In the first it requires an argument and in the second it provides a default argument of None. This is confusing when branches can be 'default' peeled but tags cannot.
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
Comparez les deux implémentations de peel dans object.c et reference.c, en vous concentrant sur la gestion de leurs arguments pour les branches et les tags. Établissez un comportement cohérent pour l’argument facultatif, puis vérifiez que les deux implémentations exposent la même interface et que le peeling par défaut reste correct.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- git, python
- Domaine
- devtools
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 38/100