Skip to content

chore(lint): Use shared eslint config#770

Open
fb55 wants to merge 4 commits intomasterfrom
shared-config
Open

chore(lint): Use shared eslint config#770
fb55 wants to merge 4 commits intomasterfrom
shared-config

Conversation

@fb55
Copy link
Owner

@fb55 fb55 commented Feb 21, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 21, 2026 23:16
Copy link

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 migrates the repo’s lint/format tooling to a shared ESLint flat-config and introduces Biome, removing the legacy .eslintrc.json/Prettier setup.

Changes:

  • Replace legacy ESLint config files with eslint.config.mjs using @feedic/eslint-config and ESLint v10.
  • Add Biome configuration and scripts; remove Prettier config/ignore file usage.
  • Apply small naming/loop refactors in source and tests to satisfy updated lint rules.

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/index.ts Renames parameters to satisfy lint rules (public API surface via generated typings).
src/index.spec.ts Updates callback param naming and replaces forEach with for...of.
package.json Switches scripts/tooling deps to ESLint v10 + Biome; removes Prettier.
package-lock.json Lockfile update for new lint/format dependency graph.
eslint.config.mjs New ESLint flat-config using shared config + TS rules.
biome.json New Biome formatter/linter configuration.
.prettierignore Removed (Prettier no longer used).
.eslintrc.json Removed (migrated to flat config).
.eslintignore Removed (ignore handling moved to gitignore/flat config).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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