Skip to content

Add CI workflow config - #57

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

Add CI workflow config#57
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: add test:types and test:unit scripts, remove the duplicate type-check script (identical to test:types) and update the deploy script to reference test:types, and downgrade typescript from 7.0.2 to 6.0.3 so it satisfies the typescript-eslint peer range.

Add eslint.config.js (previously missing), so the lint step can run.

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: add test:types and test:unit scripts, remove the
duplicate type-check script (identical to test:types) and update the
deploy script to reference test:types, and downgrade typescript from
7.0.2 to 6.0.3 so it satisfies the typescript-eslint peer range.

Add eslint.config.js (previously missing), so the lint step can run.

Built with Snoocode
Comment thread eslint.config.js
@@ -0,0 +1,68 @@
import { defineConfig } from 'eslint/config';

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

copied from three-js

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