Repository navigation
chore: replace eslint with oxlint - #4
Merged
Merged
Conversation
Swap the Next apps' eslint/eslint-config-next setup for oxlint, and add oxlint at the repo root. oxlint is the oxc linter (https://oxc.rs/docs/guide/usage/linter): one binary, no config needed, and it lints the whole tree from the root.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #
📚 Description
Replaces ESLint with oxlint (the oxc linter) across the repo, and wires linting into CI. One fast binary instead of per-app
eslint+eslint-config-next, no config file to maintain, and the whole tree lints from the root.⛳️ Current behavior
Each Next app carried the create-next-app ESLint default —
eslint+eslint-config-nextin devDeps and its owneslint.config.mjs. CI never ran lint, so nothing enforced it.🚀 New behavior
oxlintwithlint/lint:fixscripts;pnpm lintlints the whole tree.panda-app,standalone-app,monorepo/apps/web): removedeslint/eslint-config-nextand theireslint.config.mjs, repointedpnpm lintatoxlint.pnpm lintstep to thequalityjob.AGENTS.mdandCONTRIBUTING.md.No
.oxlintrc.json— defaults are enough for now; the repo lints clean today. Add one when a rule needs tuning.💣 Is this a breaking change?
No.
📝 Additional information
The large lockfile diffs are just ESLint's dependency tree being removed.