Force push does not invalidate existing approved reviews
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- github, javascript
- Bereich
- tooling
Rechercherichtung
Das Issue nennt keine Quelldatei, keinen Test und keinen Einstiegspunkt. Beginne damit, die Behandlung von Reviews und Force-Pushes in node-core-utils zu lokalisieren, und reproduziere das Szenario nodejs/node#46904; abgeschlossen ist die Aufgabe, wenn eine Genehmigung nach einem Force-Push des geprüften Branches nicht mehr zählt und dieses Verhalten durch Tests abgedeckt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
If the author force pushes to their branch after a reviewer approved it, I assumed that it would invalidate existing approved reviews, since the code that was reviewed is changed. But right now approved reviews count as valid ones, and might lead to unwanted code to be merged to main.
Example https://github.com/nodejs/node/pull/46904
➜ node git:(main) ✗ git node metadata 46904
✔ Done loading data for nodejs/node/pull/46904
----------------------------------- PR info ------------------------------------
Title url: use private properties for brand check (#46904)
Author Yagiz Nizipli <yagiz@nizipli.com> (@anonrig)
Branch anonrig:url-context -> nodejs:main
Labels semver-major, performance, whatwg-url, esm, author ready, worker, needs-ci
Commits 1
- url: use private properties for brand check
Committers 1
- Yagiz Nizipli <yagiz@nizipli.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/46904
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
--------------------------------------------------------------------------------
ℹ This PR was created on Wed, 01 Mar 2023 21:28:43 GMT
✔ Approvals: 3
✔ - Antoine du Hamel (@aduh95) (TSC): https://github.com/nodejs/node/pull/46904#pullrequestreview-1320667498
✔ - Ruben Bridgewater (@BridgeAR) (TSC): https://github.com/nodejs/node/pull/46904#pullrequestreview-1320668469
✔ - Benjamin Gruenbaum (@benjamingr): https://github.com/nodejs/node/pull/46904#pullrequestreview-1321281126
✖ This PR needs to wait 21 more hours to land
✖ GitHub CI is still running
ℹ Last Full PR CI on 2023-03-02T23:13:10Z: https://ci.nodejs.org/job/node-test-pull-request/50175/
✔ Build data downloaded
✖ Last Jenkins CI still running
- Vorherrschende Sprache
- JavaScript
- Sterne
- 313
- Forks
- 137
- Ø Merge
- 2 T. 8 Std.
- Gemergte PRs (30 T.)
- 24
Beitragsleitfaden
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 nodejs/node-core-utils
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
nodejs/node-core-utils#664 · 1 Kommentar ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 65/100
nodejs/node-core-utils#1164 ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 68/100
nodejs/node-core-utils#1160 · 3 Kommentare ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100
nodejs/node-core-utils#1151 ·
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
nodejs/node-core-utils#1063 ·
Alle Issues in nodejs/node-core-utils
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
mksglu/context-mode#1200 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
neondatabase/website#5944 ·
-
module: core
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
bigbluebutton/bigbluebutton#25849 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
jaegertracing/jaeger-ui#4506 ·