rewatch watcher: the lock-file exit path is duplicated
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 65/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- rust
- Área
- build-system
Línea de trabajo
Empieza con las dos copias del bloque de salida en rewatch/src/watcher.rs, cerca de las líneas 393 y 635, y compara sus comprobaciones del archivo de bloqueo línea por línea. Mueve la lógica compartida a una única función auxiliar y haz que ambos puntos de llamada la usen, sin cambiar el comportamiento. Se considera terminado cuando ambas rutas llaman a la función auxiliar y las pruebas de Rust existentes de rewatch (cargo test en rewatch/) siguen pasando.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- OCaml
- Estrellas
- 7.5k
- Forks
- 484
- Merge medio
- 22 h 10 min
- PR fusionados (30 d)
- 70
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de rescript-lang/rescript
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
rescript-lang/rescript#8763 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
rescript-lang/rescript#8761 ·
Los mantenedores suelen responder en 1 día
-
Issues from differential semantic drift analysis (ece8b148 to a7721303)Posiblemente ocupada @cknitt la tomó hoy. Abierto
rescript-lang/rescript#8768 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 Medio día Aptitud para principiantes 55/100
rescript-lang/rescript#8767 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 Medio día Aptitud para principiantes 72/100
rescript-lang/rescript#8766 ·
Los mantenedores suelen responder en 1 día
Todos los issues de rescript-lang/rescript
Issues similares
-
enhancement
Dificultad 2/5 Menos de una hora Aptitud para principiantes 72/100
afarber/OpenMapView#22 ·
-
[request] vsg/1.1.16Abiertoupstream update
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
conan-io/conan-center-index#31142 ·
Los mantenedores suelen responder en 1 día
-
bot-found documentation priority: P3
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
madenvel/KalinkaPlayer#313 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
setup_intel.py: sycl_version() returns "0", so setup refuses UD-IQ4_XS and UD-Q4_K_XL on IntelAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día