`cli/common/tools.js` and `tools.cjs` are not covered by `npm run typecheck`
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- Ein halber Tag
- Anfängerfreundlichkeit
- 72/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- javascript, typescript
- Bereich
- build-system
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- 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
-
good first issue needs-triage priority: medium
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
melodic-software/claude-code-plugins#7014 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
0.kind: enhancement 9.needs: package (update)
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
hpi-swa-teaching/AutoTDD#135 ·
-
bug pixi-build-r
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
prefix-dev/pixi#7229 ·
Maintainer antworten meist innerhalb von 1 Tag
-
[Bug] @loaders.gl/[email protected] is missing from npm, so [email protected] fails to installOffen
Schwierigkeit 2/5 Ein halber Tag Anfängerfreundlichkeit 70/100
visgl/loaders.gl#4208 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag