lambdalisue/vim-gina

Failed to patch

Open

#292 geöffnet am 19. Aug. 2021

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Vim Script (28 Forks)github user discovery
bughelp wanted

Repository-Metriken

Stars
 (688 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

It seems patch fails when there is no common suffix among stage, index, and worktree.

For example

|  HEAD    |  INDEX   | WORKTREE |
|+aaaaaaaa |+aaaaaaaa |          |
|+aaaaaaaa |+aaaaaaaa |          |
|          |+aaaaaaaa |+aaaaaaaa |
|          |+aaaaaaaa |+aaaaaaaa |
|+aaaaaaaa |          |          |
|+aaaaaaaa |          |          |
| XXXXXXXX | XXXXXXXX | XXXXXXXX |

works OK while the last XXXXXXXXXX are common among three but

|  HEAD    |  INDEX   | WORKTREE |
|+aaaaaaaa |+aaaaaaaa |          |
|+aaaaaaaa |+aaaaaaaa |          |
|          |+aaaaaaaa |+aaaaaaaa |
|          |+aaaaaaaa |+aaaaaaaa |
|+aaaaaaaa |          |          |
|+aaaaaaaa |          |          |

does not works while the last line is different among theee.

Contributor Guide

Research-Richtung
Untersuchen Sie den Diff Algorithmus, der in der Patch Funktion von vim gina verwendet wird, wenn es kein gemeinsames Suffix gibt. Verfolgen Sie den Codepfad, der den Patch generiert, und vergleichen Sie ihn mit dem eigenen Diff Algorithmus von git.
Tech Stack
Keine
Domain
tooling
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.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60