_-prefixed packages have tests but no test script, so yarn test-packages skips them
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 68/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- node.js, typescript
- 領域
- build-system, ci-cd, testing-qa
調査の方向性
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.
索引モデルが issue の本文から書いたものです。
説明
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.
- 主要言語
- TypeScript
- スター
- 652
- フォーク
- 64
- 平均マージ
- 8時間 45分
- マージ済み PR（30日）
- 36
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
quarto-dev/quarto のほかの issue
-
VS Code extension: `quarto preview` fails when the file path contains parentheses (path not shell-quoted)対応中かも @Bondrewd-K が 4 日前に担当しました。 オープンbug vscode
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
quarto-dev/quarto#1194 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
quarto-dev/quarto#1139 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1〜3時間 初心者へのやさしさ 90/100
quarto-dev/quarto#1138 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
quarto-dev/quarto#1137 ·
メンテナーはふだん 1 日以内に返信
-
Add dependabot?再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
quarto-dev/quarto#1135 ·
メンテナーはふだん 1 日以内に返信
quarto-dev/quarto の issue をすべて見る
似ている issue
-
freshness freshness:observed-only
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
thoroc/context-radar#280 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
-
documentation story
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
メンテナーはふだん 1 日以内に返信
-
bug category:billing priority:p3 track:fast
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
LanternOps/breeze#9223 ·
メンテナーはふだん 1 日以内に返信