Skip to content

Add GitHub Actions CI for tests and linting - #41

Merged
TimBest merged 1 commit into
mainfrom
claude/github-action-tests-linting-027qpp
Jul 30, 2026
Merged

Add GitHub Actions CI for tests and linting#41
TimBest merged 1 commit into
mainfrom
claude/github-action-tests-linting-027qpp

Conversation

@TimBest

@TimBest TimBest commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a .github/workflows/ci.yml workflow that runs on push to main and on pull requests, with two parallel jobs: lint and test
  • Add a lint script to package.json (eslint src --ext .js,.jsx,.ts,.tsx), reusing the existing eslintConfig (extends: react-app) — no new dependencies required since eslint is already pulled in transitively by react-scripts
  • Both jobs use Node 20 with NODE_OPTIONS=--openssl-legacy-provider (matches the working Netlify build config, required for react-scripts 3.4.4/webpack 4 on modern Node/OpenSSL) and cache yarn dependencies
  • test job runs Jest non-interactively via yarn test --watchAll=false with CI=true

Test plan

  • yarn install --frozen-lockfile succeeds locally
  • yarn lint passes with no errors locally
  • CI=true yarn test --watchAll=false passes locally (11 test suites, 53 tests)
  • GitHub Actions lint and test jobs pass on this PR

Generated by Claude Code

Adds a yarn lint script (eslint over src/) and a .github/workflows/ci.yml
that runs lint and test jobs on push to main and on pull requests.
@netlify

netlify Bot commented Jul 30, 2026

Copy link
Copy Markdown

Deploy Preview for imaginative-dieffenbachia-3400e7 ready!

Name Link
🔨 Latest commit 690c4cd
🔍 Latest deploy log https://app.netlify.com/projects/imaginative-dieffenbachia-3400e7/deploys/6a6bb77f3316c70008175b76
😎 Deploy Preview https://deploy-preview-41--imaginative-dieffenbachia-3400e7.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@TimBest
TimBest merged commit 99160e8 into main Jul 30, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants