Skip to content

chore: migrate from ESLint to Biome#71

Merged
evertdespiegeleer merged 4 commits into
mainfrom
chore/migrate-to-biome
Apr 24, 2026
Merged

chore: migrate from ESLint to Biome#71
evertdespiegeleer merged 4 commits into
mainfrom
chore/migrate-to-biome

Conversation

@evertdespiegeleer
Copy link
Copy Markdown
Owner

Summary

  • Replace ESLint + @typescript-eslint with Biome for linting and formatting
  • Add a minimal biome.json matching the existing style (2-space indent, single quotes, no semicolons, no trailing commas, 120-char lines); any and a few noisy rules disabled; test files get extra leniency for unused vars/params
  • Reformat the codebase via biome check --write; strip all // eslint-disable-* comments
  • Rewire npm scripts: lint β†’ biome check, lint:fix β†’ biome check --write, new format β†’ biome format --write
  • Update .vscode/extensions.json to recommend biomejs.biome

CI needs no changes β€” the existing lint job runs npm run lint, which now maps to biome check.

One stylistic casualty: Biome removes the space in methodName () β†’ methodName(). Consistent across the repo now.

Test plan

  • npm run lint passes
  • npm run build passes
  • npm test β€” all 11 tests pass
  • CI lint / build / test jobs pass on this PR

Replace ESLint + @typescript-eslint with Biome for linting and formatting.
Adds a minimal biome.json matching the existing style (2-space indent,
single quotes, no semicolons, no trailing commas) and reformats the
codebase accordingly. CI lint job works unchanged via `npm run lint`.
@github-actions
Copy link
Copy Markdown
Contributor

Test Results

 1 files  Β±0   4 suites  Β±0   0s ⏱️ Β±0s
10 tests Β±0  10 βœ… Β±0  0 πŸ’€ Β±0  0 ❌ Β±0 
11 runsβ€Š Β±0  11 βœ… Β±0  0 πŸ’€ Β±0  0 ❌ Β±0 

Results for commit da7151e. ± Comparison against base commit 3d90027.

@evertdespiegeleer evertdespiegeleer merged commit 4a0ccb8 into main Apr 24, 2026
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.

1 participant