Skip to content

Add ESLint and Prettier with CI linting#8

Merged
pmcelhaney merged 2 commits into
mainfrom
copilot/add-eslint-and-prettier
Apr 7, 2026
Merged

Add ESLint and Prettier with CI linting#8
pmcelhaney merged 2 commits into
mainfrom
copilot/add-eslint-and-prettier

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 7, 2026

Adds ESLint and Prettier to enforce consistent code style and catch issues early, with linting wired into the existing CI pipeline.

Changes

  • eslint.config.js — Flat config with @eslint/js recommended, typescript-eslint recommended, and eslint-config-prettier to avoid rule conflicts. Auto-generated files (api/types/**, api/counterfact-types/**) are excluded.
  • .prettierrc — Standard defaults (double quotes, trailing commas, semicolons).
  • package.json — Added lint and format scripts; "type": "module" for ESM config support; dev dependencies for eslint, typescript-eslint, eslint-config-prettier, and prettier.
  • .github/workflows/ci.yaml — Added npm run lint step to the existing CI job.
  • Formatted api/routes/_.context.ts, .github/workflows/create-issues.yml, and spec/petstore.yaml to satisfy Prettier.

Copilot AI linked an issue Apr 7, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Add ESLint and Prettier with recommended rules Add ESLint and Prettier with CI linting Apr 7, 2026
Copilot AI requested a review from pmcelhaney April 7, 2026 21:25
@pmcelhaney pmcelhaney marked this pull request as ready for review April 7, 2026 21:26
@pmcelhaney pmcelhaney enabled auto-merge April 7, 2026 21:27
@pmcelhaney pmcelhaney added this pull request to the merge queue Apr 7, 2026
Merged via the queue into main with commit 298b2c7 Apr 7, 2026
4 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.

Add ESLint and Prettier

2 participants