Skip to content

Comments

Add CI/CD workflows and ESLint configuration#3

Merged
dortort merged 1 commit intomainfrom
claude/add-github-actions-workflows-xm7Mj
Feb 6, 2026
Merged

Add CI/CD workflows and ESLint configuration#3
dortort merged 1 commit intomainfrom
claude/add-github-actions-workflows-xm7Mj

Conversation

@dortort
Copy link
Owner

@dortort dortort commented Feb 6, 2026

Summary

This PR introduces automated code quality and testing infrastructure by adding GitHub Actions workflows and ESLint configuration to the project.

Key Changes

  • GitHub Actions Workflows: Added two new CI/CD workflows
    • lint.yml: Runs ESLint on Node.js 20 for every push to main and pull requests
    • test.yml: Runs build and tests on Node.js 20 and 22 using a matrix strategy
  • ESLint Configuration: Added eslint.config.js with recommended rules for JavaScript and TypeScript, ignoring dist/ and examples/ directories
  • Dependencies: Added ESLint and TypeScript ESLint packages to devDependencies
  • Code Cleanup: Removed unused imports in src/cli/index.ts and src/gen/index.ts to comply with linting rules

Implementation Details

  • ESLint is configured using the new flat config format with @eslint/js and typescript-eslint packages
  • CI workflows use Node.js 20 as the baseline with Node.js 22 included in the test matrix for forward compatibility
  • npm cache is leveraged in both workflows to improve CI performance

https://claude.ai/code/session_01GWGvS1VjSY6rNR2chqbFJv

Set up ESLint with typescript-eslint for the project (the lint script
was defined but eslint was not installed) and add CI workflows that run
on pushes and PRs to main. The test workflow runs against Node 20 and 22.

https://claude.ai/code/session_01GWGvS1VjSY6rNR2chqbFJv
@dortort dortort marked this pull request as ready for review February 6, 2026 14:06
@dortort dortort merged commit 1721683 into main Feb 6, 2026
3 checks passed
@dortort dortort deleted the claude/add-github-actions-workflows-xm7Mj branch February 7, 2026 19:48
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