_-prefixed packages have tests but no test script, so yarn test-packages skips them
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 68/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- node.js, typescript
- Ambito
- build-system, ci-cd, testing-qa
Direzione di ricerca
Compare the package.json files in packages/_annotated-json, packages/_mapped-string, packages/_tidyverse-errors, and packages/_json-validator with their published siblings. Check the Node version in .nvmrc and the CI workflow at .github/workflows/test.yaml, then run yarn test-packages. Done means the intended tests run and pass, with the unused _json-validator package's fate decided based on its imports.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Bug
These workspaces have tests (test/basic.test.ts) but no scripts at all in their package.json:
packages/_annotated-jsonpackages/_mapped-stringpackages/_tidyverse-errorspackages/_json-validator
yarn test-packages runs turbo run test --filter='./packages/*', which only runs workspaces that define a test script, so these tests are silently skipped locally and in CI (.github/workflows/test.yaml). The _-prefixed packages are what the shipped extension actually uses. Their published, non-prefixed siblings (packages/annotated-json, etc.) do have "test": "node --import tsx --test test/*.test.ts".
Fix
Add the same test script to each _ package, and make sure tsx is declared as a devDependency (it is in the plan's inventory). Then run yarn test-packages and fix any tests that fail after going unrun for a while.
Note that node --import tsx needs Node ≥18.18/20.6. .nvmrc currently pins 18.12.1, which can't run it (the Node 24 bump in the upgrade plan fixes that). CI uses node-version: latest, so it works there.
Separately, _json-validator isn't imported anywhere. Consider deleting it rather than wiring up its tests.
Found during the dependency-upgrade research.
- Lingua principale
- TypeScript
- Stelle
- 652
- Fork
- 64
- Merge medio
- 8h 45m
- PR unite (30g)
- 36
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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 quarto-dev/quarto
-
VS Code extension: `quarto preview` fails when the file path contains parentheses (path not shell-quoted)Forse già presa @Bondrewd-K l’ha presa 4 giorni fa. Apertabug vscode
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
quarto-dev/quarto#1194 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
quarto-dev/quarto#1139 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 90/100
quarto-dev/quarto#1138 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
quarto-dev/quarto#1137 ·
I maintainer di solito rispondono entro 1 giorno
-
Add dependabot?Forse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
quarto-dev/quarto#1135 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di quarto-dev/quarto
Issue simili
-
documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
components-web-app/docs#260 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 65/100
521xueweihan/HelloGitHub#4025 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
ctrf-io/playwright-ctrf-json-reporter#117 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
voidzero-dev/void#478 ·
I maintainer di solito rispondono entro 1 giorno