Skip to content

Conversation

@L-Mario564
Copy link
Collaborator

This PR aims to replace ESLint in favor of Oxlint and fix an issue with Dprint.

List of changes:

  • Replaced ESLint with Oxlint. Many rules were very easy to port over; however, there are a few exceptions:
    • unused-imports/no-unused-imports. unusued-imports ESLint plugin is not available for Oxlint, but it seems Oxlint's no-unused-vars rule already handles this plugin and rule's behavior.
    • import/no-useless-path-segments and import/newline-after-import are not yet implemented.
    • typescript/no-unused-vars is not implemented, which allows TS types to go unused without warning.
    • no-instanceof/no-instanceof plugin is not available. I implemented this rule's behavior in the drizzle-internal plugin to get similar behavior.
    • drizzle-internal is now a Oxlint plugin and ported drizzle-internal/require-entity-kind to Oxlint.
  • Changed some scripts in the main package.json.
  • Committed .vscode folder. This alows VSCode to pick on the fact that this repo uses Dprint instead of Prettier which will eliminate the need to manually set the editor.defaultFormatter option in the global VSCode settings.
  • Added a pre-commit hook with Husky and lint-staged that checks formatting and linting before committing.
  • Any other change in unrelated files were done to conform with Oxlint's linting behavior.
  • Removed everything related to ESLint and Prettier.

@AlexBlokh AlexBlokh merged commit 77347fc into drizzle-team:alternation-engine Oct 20, 2025
1 of 32 checks passed
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