Add a test runner, a lint step and a pull_request CI check
還沒有人認領這個 Issue。
評估
- 難度
- 3/5
- 預估耗時
- 1-2 天
- 新手友好度
- 65/100
- Issue 類型
- 功能
- 描述清晰度
- 描述清楚
- 活躍度
- 活躍
- 技術堆疊
- eslint, github-actions, typescript, vite
- 領域
- build-system, ci-cd, testing
研究方向
Read package.json and .github/workflows/deploy.yml to understand the existing scripts and deployment workflow; check tsconfig.json for the current type-check scope. Add the requested test, lint and pull request checks, keeping deployment gated to main, then run npm test, npm run lint and npm run build. Done when all checks pass and pull requests receive a required check.
由索引模型根據 Issue 內容生成。
描述
Summary
There is no test framework, no linter and no check before code reaches main. The only workflow deploys on push if tsc and vite build pass.
Where
package.json:8-14(scripts: dev, build, preview, typecheck, new-game).github/workflows/deploy.yml:3-6,.github/workflows/deploy.yml:26-27tsconfig.json:17(scripts/new-game.mjsis not type-checked)
Impact
A logic regression in rules code, trophy unlocks or save handling ships with no automated signal, and pull requests get no feedback.
Suggested fix
- Add
vitestwith"test": "vitest run"and a separatevitest.config.tsusingenvironment: 'node', so the PWA plugin and multi-page inputs are not loaded - Add ESLint with typescript-eslint and a
lintscript - Add a
pull_requesttrigger that runs install, lint, test and build only (deploy stays gated to main) - Run
npm testbeforenpm run buildin the deploy workflow
Acceptance criteria
-
npm testandnpm run lintexist and pass - Pull requests show a required check
Getting started
- Relevant files:
package.json,.github/workflows/deploy.yml,tsconfig.json - Run locally:
npm install, thennpm run dev - Checks:
npm run typecheckandnpm run build(there is no test suite yet) - Related docs:
README.md
Filed by repo-audit/full-audit at b001f5d. Severity: medium. Effort: M. audit-key: testing|package.json|scripts.test
- 主要語言
- TypeScript
- 星號
- 3
- 分支
- 3
- PR 合併指標
- 30 天內沒有已合併 PR
環境準備
這個專案沒有提供開發容器、Dockerfile 或貢獻指南，環境需要你自己搭建：先看它的 README，通用步驟見我們的新手貢獻指南。
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
TMHSDigital/GH-Arcade 的其他 Issue
-
enhancement good first issue severity: low
難度 2/5 1-3 小時 新手友好度 82/100
TMHSDigital/GH-Arcade#50 ·
-
enhancement good first issue severity: medium
難度 2/5 1-3 小時 新手友好度 86/100
TMHSDigital/GH-Arcade#47 ·
-
documentation good first issue severity: low
難度 2/5 1-3 小時 新手友好度 78/100
TMHSDigital/GH-Arcade#34 ·
-
documentation good first issue severity: low
難度 2/5 1-3 小時 新手友好度 78/100
TMHSDigital/GH-Arcade#33 ·
-
dependencies good first issue severity: low
難度 2/5 1-3 小時 新手友好度 85/100
TMHSDigital/GH-Arcade#27 ·
查看 TMHSDigital/GH-Arcade 的全部 Issue
相似的 Issue
-
automated-triage type/bug type/ci
難度 2/5 1-3 小時 新手友好度 78/100
維護者通常 1 天內回覆
-
📦 expo-router Android contributor: external needs review sdk-58 severity: medium
難度 2/5 1-3 小時 新手友好度 78/100
維護者通常 1 天內回覆
-
documentation good first issue
難度 2/5 1-3 小時 新手友好度 84/100
asasemahmed/openboard#68 ·
維護者通常 1 天內回覆
-
bug good first issue
難度 2/5 1-3 小時 新手友好度 72/100
-
Document PowerShell shell semantics and read-only Bash deduplication in Built-in Tools reference未關閉documentation
難度 2/5 1-3 小時 新手友好度 78/100