branch upstream_name InvalidRefSpec error on literal remote values for branch
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
when using hub (https://hub.github.com) to checkout pull requests, it generates a config entry for the branch with the direct reference to the remote git url instead of adding in a remote. This seems to cause some confusion for pygit, specifically iy will end up throwing an InvalidSpecError as pygit tries to follow the remote as a name instead of being to handle a literal git repo value. I'd suggest in such a case pygit simply return the remote name.
https://github.com/libgit2/pygit2/blob/master/src/branch.c#L245
$ branch
<_pygit2.Branch object at 0x10d1197b0>
$ branch.upstream_name
*** _pygit2.InvalidSpecError: 'https://github.com/steveparker-1984/cloud-custodian.git' is not a valid remote name.
entry in .git/config
[branch "fail-on-assume-role-error"]
remote = https://github.com/steveparker-1984/cloud-custodian.git
merge = refs/heads/fail-on-assume-role-error
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
Beginne damit, src/branch.c ungefähr bei Zeile 245 zu untersuchen und die gezeigte Konfiguration mit einem literalen HTTPS-Wert in branch.remote nachzustellen. Verfolge die Verarbeitung von branch.upstream_name für diesen Wert; fertig ist es, wenn nicht mehr InvalidSpecError ausgelöst wird und das angeforderte Verhalten für den Remote-Namen befolgt wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- git, python
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100