Skip to content

Migrate to oxlint#64

Closed
ocavue wants to merge 1 commit into
changesets:vitestfrom
ocavue-forks:ocavue/oxlint
Closed

Migrate to oxlint#64
ocavue wants to merge 1 commit into
changesets:vitestfrom
ocavue-forks:ocavue/oxlint

Conversation

@ocavue
Copy link
Copy Markdown

@ocavue ocavue commented May 18, 2026

Migrate from ESLint to oxlint.

This is a stack PR based on #62 because I need to test the vitest oxlint config. I will make this PR ready for review once #62 is merged into main.

@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedoxlint@​1.65.0991009196100
Addedoxlint-tsgolint@​0.22.11001009795100

View full report

Comment thread tsconfig.json
"extends": "./tsconfig.base.json",
"compilerOptions": {
"types": ["node"],
"rootDir": ".",
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Note for reviewers:

oxlint-tsgolint requests these changes. We will need them anyway when we update TS5 to TS6.

Comment thread .eslintrc.js
Comment on lines -10 to -13
globals: {
React: true,
JSX: true,
},
Copy link
Copy Markdown
Author

@ocavue ocavue May 18, 2026

Choose a reason for hiding this comment

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

Note for reviewers:

We're not using React in this project.

Comment thread .oxlintrc.json
Comment on lines +8 to +16
"plugins": [
"eslint",
"typescript",
"unicorn",
"oxc",
"import",
"node",
"vitest"
],
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Note for reviewers:

I enabled all default plugins, plus some other useful plugins. See https://oxc.rs/docs/guide/usage/linter/plugins.html#supported-plugins for a full list of plugins.

@bluwy bluwy deleted the branch changesets:vitest May 18, 2026 09:38
@bluwy bluwy closed this May 18, 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.

2 participants