rewatch watcher: the lock-file exit path is duplicated
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 65/100
- Tipo di issue
- Refactoring
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- rust
- Ambito
- build-system
Direzione di ricerca
Inizia dalle due copie del blocco di uscita in rewatch/src/watcher.rs, vicino alle righe 393 e 635, e confronta riga per riga i controlli sul file di lock. Sposta la logica condivisa in un'unica funzione di supporto e fai in modo che entrambi i punti di chiamata la usino, senza cambiare il comportamento. È completato quando entrambi i percorsi chiamano la funzione di supporto e i test Rust esistenti di rewatch (cargo test in rewatch/) continuano a passare.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- OCaml
- Stelle
- 7.5k
- Fork
- 484
- Merge medio
- 19h 20m
- PR unite (30g)
- 96
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di rescript-lang/rescript
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
rescript-lang/rescript#8806 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
rescript-lang/rescript#8798 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 Mezza giornata Idoneità per principianti 58/100
rescript-lang/rescript#8757 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 40/100
rescript-lang/rescript#8753 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 22/100
rescript-lang/rescript#8733 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di rescript-lang/rescript
Issue simili
-
needs-triage Platform(Default)
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
atlassian-labs/compiled#1960 ·
I maintainer di solito rispondono entro 1 giorno
-
Cached wheel reuse fails when the build and metadata fallback share an output directoryForse già presa @harbinresearcher l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
pypa/pyproject-hooks#240 · 1 commento ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
servo/mozjs#831 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Declare Python 3.15 supportApertaenhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 6 giorni