Skip to content

chore: replace half-baked eslint setup with oxlint#610

Draft
serhalp wants to merge 1 commit intomainfrom
serhalp/eslint-to-oxlint
Draft

chore: replace half-baked eslint setup with oxlint#610
serhalp wants to merge 1 commit intomainfrom
serhalp/eslint-to-oxlint

Conversation

@serhalp
Copy link
Member

@serhalp serhalp commented Mar 4, 2026

We had set up eslint in this repo after the fact, but in the interest of time we hadn't quite set it up correctly, we hadn't curated plugins and rules to enable, and we had suppressed hundreds of errors.

By now, oxlint is stable and solid: https://oxc.rs/docs/guide/usage/linter.html. And it's extremely fast. So this PR revamps the linting setup to use oxlint.

oxlint has the concept of rule "categories". This PR only enables all the "correctness" rules. These all catch definite errors. We can always add more later.

Tip

The first time you work in a codebase that uses oxlint, you may need to install a new extension in your IDE: https://oxc.rs/docs/guide/usage/linter/editors.html.

@coderabbitai
Copy link

coderabbitai bot commented Mar 4, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1c23c630-6ac8-47a8-84c0-27163f237a83

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch serhalp/eslint-to-oxlint

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@serhalp serhalp force-pushed the serhalp/eslint-to-oxlint branch 2 times, most recently from e07294f to bd9b714 Compare March 4, 2026 18:22
@serhalp serhalp force-pushed the serhalp/eslint-to-oxlint branch from bd9b714 to 2178fa8 Compare March 4, 2026 18:24
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