guyzmo/git-repo

config wizard does not work if ~/.config/git/config contains include statements

Open

#153 geöffnet am 7. Mai 2017

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (86 Forks)batch import
backlogbughelp wanted

Repository-Metriken

Stars
 (837 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Steps to reproduce:

  • Make sure ~/.git does not exist
  • Create ~/.config/git/config` with a least two includes, like so:
[include]
path = /path/to/1
path = /path/to/2

Run git-repo config, do the oauth dance.

Nothing is stored.

PS: I think this is a bug in the code that reads and writes git config. If I were you, I'll just run git config --get foo.bar and git config --global foo.bar 42 instead of using a re-implementation of git config parsing in Python.

Contributor Guide

Research-Richtung
Untersuche das Konfigurationslese /schreibmodul im Repository (wahrscheinlich ein Python Modul). Das Problem schlägt vor, das benutzerdefinierte Parsing durch Aufrufe von `git config get` und `git config global` zu ersetzen. Suche nach Stellen, an denen Include Direktiven nicht korrekt behandelt werden. Erwäge, den vorgeschlagenen Ansatz zu implementieren oder das Parsing zu beheben, um mehrere Includes zu verarbeiten.
Tech Stack
python
Domain
clitooling
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Pythongit config basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60