`peel` arguments inconsistent.
Offen
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 1.7k
- Forks
- 408
- Ø Merge
- 2 T. 57 Min.
- Gemergte PRs (30 T.)
- 7
Beschreibung
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.
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
Vergleiche die beiden peel-Implementierungen in object.c und reference.c und konzentriere dich dabei auf ihre Argumentbehandlung für Branches und Tags. Lege ein einheitliches Verhalten für das optionale Argument fest und überprüfe anschließend, dass beide Implementierungen dieselbe Schnittstelle bereitstellen und das standardmäßige Peeling weiterhin korrekt funktioniert.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- git, python
- Bereich
- devtools
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 38/100