Skip to content

chore(lint): Use shared eslint config#1314

Merged
fb55 merged 8 commits intomasterfrom
shared-config
Mar 4, 2026
Merged

chore(lint): Use shared eslint config#1314
fb55 merged 8 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 pull request migrates from the legacy ESLint configuration format (.eslintrc.json) to the modern flat config format (eslint.config.mjs) using a shared configuration from @feedic/eslint-config. The changes also introduce Biome as an additional formatter/linter and upgrade ESLint from v8 to v10.

Changes:

  • Upgraded ESLint from v8.57.1 to v10.0.1 and adopted flat config format
  • Added shared ESLint configuration via @feedic/eslint-config package
  • Introduced Biome (v2.4.4) for additional formatting/linting
  • Applied automated code fixes (parentheses for operator precedence, modern array methods)

Reviewed changes

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

Show a summary per file
File Description
package.json Updated dependencies (ESLint v10, removed Prettier, added Biome and shared config), reorganized scripts, added "type": "commonjs" field
package-lock.json Dependency lock file updates reflecting new package versions and added/removed dependencies
eslint.config.mjs New flat ESLint configuration file using shared config and TypeScript rules
biome.json New Biome configuration for formatting and linting
.eslintrc.json Removed legacy ESLint configuration
.eslintignore Removed (replaced by config in flat config file)
.prettierignore Removed (Prettier replaced by Biome)
src/index.ts Added parentheses around modulo operations for clarity, renamed loop variables
src/index.spec.ts Modernized code using Array.from() and .entries()

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

@fb55 fb55 merged commit 62cc132 into master Mar 4, 2026
13 checks passed
@fb55 fb55 deleted the shared-config branch March 4, 2026 22:00
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