Skip to content

Align CI rules - #94

Open
Krakabek wants to merge 5 commits into
mainfrom
dr-setup-ci
Open

Align CI rules#94
Krakabek wants to merge 5 commits into
mainfrom
dr-setup-ci

Conversation

@Krakabek

Copy link
Copy Markdown
Contributor

💸 TL;DR

Aligns this repo's CI to https://github.com/reddit/devvit-template-bare/blob/main/.github/workflows/ci.yaml

📜 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 2 commits July 31, 2026 15:53
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.

Built with Snoocode
@Krakabek
Krakabek requested a review from stephenoid July 31, 2026 13:57
@Krakabek Krakabek self-assigned this Jul 31, 2026
danil.radkovskyi added 3 commits July 31, 2026 16:09
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, delete the committed
package-lock.json, and gitignore it (the workflow uses npm install).

Built with Snoocode
The type-check script was removed but the deploy script still referenced
it; update deploy to use test:types (identical tsc --build).

Built with Snoocode
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