rewatch watcher: the lock-file exit path is duplicated
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 65/100
- Issue-Typ
- Refactoring
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- rust
- Bereich
- build-system
Rechercherichtung
Beginne mit den zwei Kopien des Exit-Blocks in rewatch/src/watcher.rs, etwa bei Zeile 393 und 635, und vergleiche ihre Lock-File-Prüfungen Zeile für Zeile. Verschiebe die gemeinsame Logik in eine einzige Hilfsfunktion und lass beide Aufrufstellen diese verwenden, ohne das Verhalten zu ändern. Fertig ist, wenn beide Pfade die Hilfsfunktion aufrufen und die bestehenden Rust-Tests von rewatch (cargo test in rewatch/) weiterhin bestehen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Found by differential semantic drift analysis from ece8b148 to a7721303.
What is broken
In rewatch/src/watcher.rs, the block that removes the lock file and exits with "Exiting... (lockfile removed)" appears twice: at L393 and, added by #8730, at L635, each with its own check for the lock file. A future change to the exit behaviour has to be made in both places.
Possible cause (speculation only)
A guess: #8730 fixed the full-rebuild path by copying the existing exit block into it, rather than factoring it into a shared function.
- Vorherrschende Sprache
- OCaml
- Sterne
- 7.5k
- Forks
- 484
- Ø Merge
- 19 Std. 46 Min.
- Gemergte PRs (30 T.)
- 81
Entwicklungsumgebung
Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
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 rescript-lang/rescript
-
Schwierigkeit 3/5 Ein halber Tag Anfängerfreundlichkeit 58/100
rescript-lang/rescript#8757 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 40/100
rescript-lang/rescript#8753 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 22/100
rescript-lang/rescript#8733 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
rescript-lang/rescript#8727 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
rescript-lang/rescript#8726 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in rescript-lang/rescript
Ähnliche Issues
-
PacReq: mangowcOffenpackage request
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
pacstall/pacstall-programs#9475 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 71/100
yjh051108/dsh-routing-suite#227 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
JoviDeCroock/pracht#432 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
munich-quantum-toolkit/workflows#492 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag