Skip to content

chore: migrate to Oxlint - #1493

Open
sapphi-red wants to merge 2 commits into
mainfrom
chore/oxlint
Open

sapphi-red wants to merge 2 commits into
mainfrom
chore/oxlint

Conversation

@sapphi-red

Copy link
Copy Markdown
Member

Description

Migrate to Oxlint from ESLint. Also enable type aware rules.

I've tried to keep the existing enabled rules while keeping the config simple enough.

@@ -1,5 +1,5 @@
/* global window */
/* eslint-disable eqeqeq, prefer-const, @typescript-eslint/no-empty-function */
/* oxlint-disable eqeqeq, prefer-const, no-empty-function, no-unused-vars, typescript/no-implied-eval */

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At some point we should split the React part and have linting disabled, and have our own part in TS and bundle that (but maybe not that simple given how it's written to be a single module)

@ArnaudBarre

Copy link
Copy Markdown
Member

Thanks, I add this on my todo list, better than merging the dependabot eslint 10 PR

@bluwy bluwy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Looking forward to migrate more of our repos to oxlint soon.

This branch has not been deployed

No deployments
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.

4 participants