`cli/common/tools.js` and `tools.cjs` are not covered by `npm run typecheck`
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- Une demi-journée
- Accessibilité débutants
- 72/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- javascript, typescript
- Domaine
- build-system
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- OCaml
- Étoiles
- 7.5k
- Forks
- 484
- Merge moyen
- 19 h 46 min
- PR mergées (30 j)
- 81
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de rescript-lang/rescript
-
Difficulté 3/5 Une demi-journée Accessibilité débutants 58/100
rescript-lang/rescript#8757 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 40/100
rescript-lang/rescript#8753 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 22/100
rescript-lang/rescript#8733 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
rescript-lang/rescript#8727 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
rescript-lang/rescript#8726 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de rescript-lang/rescript
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Homebrew/homebrew-core#317477 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 1-3 heures Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
ci
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
aieng-abdullah/production-rag-assistant#154 ·
Les mainteneurs répondent en général sous 1 jour
-
PacReq: mangowcOuvertepackage request
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
pacstall/pacstall-programs#9475 ·
Les mainteneurs répondent en général sous 2 jours
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
alunduil/zellij-claude-pair#99 ·
Les mainteneurs répondent en général sous 1 jour