microsoft / microsoft/vscode-pull-request-github

Support line-level review progress within a PR diff

Offen
#8,620 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

feature-request
Vorherrschende Sprache
TypeScript
Sterne
2.6k
Forks
795
Ø Merge
1 T. 4 Std.
Gemergte PRs (30 T.)
46

Beschreibung

Request

Please support line-level review progress within a PR diff.

Reasoning

For large changed files, file-level viewed/unviewed is too coarse. I often review a file in multiple passes and want a way to distinguish the lines I already reviewed from the lines I still need to look at.

Possible behavior
  • mark selected lines or ranges as reviewed
  • visually de-emphasize already-reviewed lines
  • persist the state locally for the current PR / head SHA

A local-only feature would already be useful.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Es werden keine Dateien oder Tests genannt. Beginne damit, den bestehenden dateiweiten Status „angesehen/nicht angesehen“ in der PR-Diff-UI und seinen Persistenzpfad zu ermitteln. Für das Issue muss ein festgelegtes Design für die Auswahl von Bereichen, die Anzeige überprüfter Zeilen und die Begrenzung des gespeicherten Status auf eine PR und einen head SHA vorliegen, bevor die Implementierung als abgeschlossen betrachtet werden kann.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
developer-experience
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Ruhig
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
38/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.