Skip to content

chore(lint): add ESLint v9 flat config + Prettier, format codebase#24

Merged
jbarson merged 1 commit into
mainfrom
chore/18-eslint-prettier
Aug 10, 2025
Merged

chore(lint): add ESLint v9 flat config + Prettier, format codebase#24
jbarson merged 1 commit into
mainfrom
chore/18-eslint-prettier

Conversation

@jbarson
Copy link
Copy Markdown
Owner

@jbarson jbarson commented Aug 10, 2025

and resolve lint errors for #18

@jbarson jbarson requested a review from Copilot August 10, 2025 16:40

This comment was marked as outdated.

Copy link
Copy Markdown
Owner Author

@jbarson jbarson left a comment

Choose a reason for hiding this comment

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

Addressed review comment:

  • Removed .eslintrc.cjs from the ignores array in eslint.config.mjs so it gets linted.
  • Left .eslintrc.cjs as a comment-only file to avoid no-undef and keep it purely informational.
  • Added a targeted override in eslint.config.mjs to disable the Prettier rule for eslint.config.mjs and .eslintrc.cjs (avoids a noisy spacing warning while other rules still apply).

Ran npm run lint locally — zero errors/warnings now.

@jbarson jbarson requested a review from Copilot August 10, 2025 16:53
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds ESLint v9 flat configuration and Prettier to enforce code style consistency and implements a comprehensive formatting update across the entire codebase. The changes standardize quote usage, apply consistent code formatting, and establish proper linting infrastructure.

  • Adds ESLint v9 flat config with TypeScript support and Prettier integration
  • Converts all single quotes to double quotes throughout the codebase
  • Applies consistent formatting with proper line breaks and spacing

Reviewed Changes

Copilot reviewed 11 out of 13 changed files in this pull request and generated no comments.

Show a summary per file
File Description
eslint.config.mjs New ESLint v9 flat configuration with TypeScript and Prettier rules
.prettierrc.json Prettier configuration with double quotes and 100-char line width
.eslintrc.cjs Deprecated legacy ESLint config placeholder
package.json Added lint and format scripts plus ESLint/Prettier dependencies
vite.config.js Quote style normalization to double quotes
scripts/systemsList.ts Multiline object formatting for better readability
scripts/jumpLinks.ts Quote style conversion from single to double quotes
scripts/app.ts Quote normalization and multiline formatting improvements
scripts/app.test.ts Quote style and import formatting updates
index.html HTML formatting improvements and DOCTYPE normalization
css/style.css CSS formatting improvements and color code normalization

@jbarson jbarson merged commit d3ed0ee into main Aug 10, 2025
@jbarson jbarson deleted the chore/18-eslint-prettier branch August 10, 2025 16:56
jbarson added a commit that referenced this pull request Mar 28, 2026
jbarson added a commit that referenced this pull request Mar 28, 2026
security: mitigate dependabot alert #24 by updating flatted and brace-expansion
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