`cli/common/tools.js` and `tools.cjs` are not covered by `npm run typecheck`
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- Medio día
- Aptitud para principiantes
- 72/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- javascript, typescript
- Área
- build-system
Línea de trabajo
The gap is in the include array of tsconfig.json, which lists "cli/*" and so misses the nested cli/common/ directory. Start by running npm run typecheck and confirming that cli/common/tools.js and tools.cjs are absent from the checked files, then compare with bins.cjs, which is checked only because bins.js imports it. Done means the typecheck covers both files and either passes or surfaces type errors that are then fixed.
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
cli/common/tools.js and tools.cjs, added by #8694, start with // @ts-check, but tsconfig.json includes "cli/*", which does not reach cli/common/. bins.cjs is checked only because bins.js imports it; nothing that is checked imports tools.*. So npm run typecheck never type-checks them, despite their @ts-check header.
Related: #8708 (listed in #8712) made npm run typecheck pass; these two files were added later by #8694 and fall outside it.
Possible cause (speculation only)
A guess: the include pattern was written when all CLI entry points lived directly in cli/, and the new subdirectory files were assumed to be covered.
- 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 65/100
rescript-lang/rescript#8765 ·
Los mantenedores suelen responder en 1 día
-
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
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