Skip to content

Add CI workflow config - #72

Draft
Krakabek wants to merge 1 commit into
mainfrom
dr-setup-ci
Draft

Add CI workflow config#72
Krakabek wants to merge 1 commit into
mainfrom
dr-setup-ci

Conversation

@Krakabek

Copy link
Copy Markdown
Contributor

Add .github/workflows/ci.yaml matching the bare template (with node-version 22.20.0, since this template uses vite 8 / rolldown which require node >=22.12.0).

package.json: wire up the scripts the workflow expects. This template already uses vitest, so test:unit runs 'vitest run'; add test:types, make the aggregate test script match bare, remove the duplicate type-check script, and point deploy at the test:* scripts. Bump engines.node to >=22.12.0 to match vite's requirement.

Also add the missing 'globals' devDependency: eslint.config.js imports it but it was not declared, so lint failed on a clean install.

Closes #

💸 TL;DR

📜 Details

Design Doc

Jira

🧪 Testing Steps / Validation

✅ Checks

  • CI tests (if present) are passing
  • Adheres to code style for repo
  • Contributor License Agreement (CLA) completed if not a Reddit employee

Add .github/workflows/ci.yaml matching the bare template (with
node-version 22.20.0, since this template uses vite 8 / rolldown which
require node >=22.12.0).

package.json: wire up the scripts the workflow expects. This template
already uses vitest, so test:unit runs 'vitest run'; add test:types,
make the aggregate test script match bare, remove the duplicate
type-check script, and point deploy at the test:* scripts. Bump
engines.node to >=22.12.0 to match vite's requirement.

Also add the missing 'globals' devDependency: eslint.config.js imports
it but it was not declared, so lint failed on a clean install.

Built with Snoocode
@Krakabek Krakabek self-assigned this Jul 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants