Skip to content

Add CI workflow config - #64

Merged
Krakabek merged 3 commits into
mainfrom
dr-setup-ci
Aug 3, 2026
Merged

Add CI workflow config#64
Krakabek merged 3 commits into
mainfrom
dr-setup-ci

Conversation

@Krakabek

Copy link
Copy Markdown
Contributor

Add .github/workflows/ci.yaml matching the bare template, and add the test:types and test:unit scripts to package.json so the workflow's typecheck, lint, unit-test, and build steps all run.

Downgrade typescript from 7.0.2 to 6.0.3 so it satisfies the typescript-eslint peer range; typescript-eslint has no released version supporting TS 7.0 yet, and the previous pin broke npm install and lint.

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

danil.radkovskyi added 3 commits July 31, 2026 16:03
Add .github/workflows/ci.yaml matching the bare template, and add the
test:types and test:unit scripts to package.json so the workflow's
typecheck, lint, unit-test, and build steps all run.

Downgrade typescript from 7.0.2 to 6.0.3 so it satisfies the
typescript-eslint peer range; typescript-eslint has no released version
supporting TS 7.0 yet, and the previous pin broke npm install and lint.

Built with Snoocode
Bump the CI node-version from 22.6.0 to 22.20.0 so the build works:
this template uses vite 8 / rolldown, which require node
^20.19.0 || >=22.12.0, so the build failed on 22.6.0 with a missing
rolldown native binding. Also bump engines.node to >=22.12.0 to match.

Built with Snoocode
Remove the type-check script (identical to test:types) and point the
deploy script at test:types instead, to avoid a duplicate script.

Built with Snoocode
@Krakabek
Krakabek marked this pull request as ready for review August 3, 2026 10:19
@Krakabek
Krakabek merged commit 2b6a58e into main Aug 3, 2026
1 check passed
@Krakabek
Krakabek deleted the dr-setup-ci branch August 3, 2026 10:19
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