Sync bot crushing partial reviews
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 30/100
Rechercherichtung
Es werden keine Dateien, Tests oder Einstiegspunkte genannt. Beginne damit, den Branch- und Pull-Request-Umgang des Sync-Bots zu lokalisieren, und reproduziere dann eine Synchronisierung mit einem teilweise durchgeführten Review; fertig ist es, wenn die Synchronisierung nicht mehr per force-push über bereits reviewte Arbeit hinwegschreibt und den Review-Kontext bewahrt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Hi @iliakan
I discovered this in the Spanish repo,
but happens in the other repos too.
...
Previously, the bot used different branches for sync, closing the old PR and generating a new one over a new branch. Partial reviews were preserved in the closed PR.
Now, the bot closes the old PR but always uses the same branch, doing a force push on it that destroys any partial reviews.
- Vorherrschende Sprache
- HTML
- Sterne
- 25.5k
- Forks
- 4k
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus javascript-tutorial/en.javascript.info
-
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 88/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 92/100
-
Keyboard events: keydown filtering example can't block IME-composed input (Korean/Japanese/Chinese) Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
javascript-tutorial/en.javascript.info#3978 · 1 Kommentar ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Alle Issues in javascript-tutorial/en.javascript.info
Ähnliche Issues
-
Add dependabot Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 75/100
-
oblt-aw/detector/security
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
-
Type: AVM :a: :v: :m: Type: Bug :bug:
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
Azure/bicep-registry-modules#7386 · 1 Kommentar ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100