Skip to content

Remove ESLint rules now covered by Biome#15

Merged
IvanKiral merged 2 commits intomasterfrom
change_to_biome_rules
Jan 7, 2026
Merged

Remove ESLint rules now covered by Biome#15
IvanKiral merged 2 commits intomasterfrom
change_to_biome_rules

Conversation

@IvanKiral
Copy link
Copy Markdown
Contributor

@IvanKiral IvanKiral commented Jan 5, 2026

Motivation

Remove ESLint rules that are now covered by Biome linter to avoid duplicate linting and reduce configuration overhead.

Rules Removed

TypeScript rules (from index.js)

React rules (from react.js)

Biome rules still pending (PRs not merged yet)

The following rules are tracked for future removal once their Biome equivalents are released:

  • @typescript-eslint/prefer-string-starts-ends-with - PR #8582
  • @typescript-eslint/prefer-nullish-coalescing - Issue #8043
  • react/jsx-no-bind - PR #7410

Checklist

  • Code follows coding conventions held in this repo
  • Automated tests have been added
  • Tests are passing
  • Docs have been updated (if applicable)
  • Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test

No manual testing required. This change removes rules from ESLint configuration. Verify by running ESLint to ensure no configuration errors.

@IvanKiral IvanKiral requested a review from JiriLojda as a code owner January 5, 2026 14:25
@IvanKiral IvanKiral force-pushed the change_to_biome_rules branch from da1aa92 to c350fa4 Compare January 5, 2026 14:37
@IvanKiral IvanKiral force-pushed the change_to_biome_rules branch from fff9f12 to bed0314 Compare January 7, 2026 10:53
@IvanKiral IvanKiral force-pushed the change_to_biome_rules branch from bed0314 to db4c188 Compare January 7, 2026 10:54
@IvanKiral IvanKiral merged commit b8ddcc0 into master Jan 7, 2026
@IvanKiral IvanKiral deleted the change_to_biome_rules branch January 7, 2026 14:09
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