Skip to content

migrate to oxlint#133

Merged
Light2Dark merged 1 commit into
mainfrom
sham/move-to-oxlint
May 29, 2026
Merged

migrate to oxlint#133
Light2Dark merged 1 commit into
mainfrom
sham/move-to-oxlint

Conversation

@Light2Dark

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings May 29, 2026 14:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR migrates the project’s local linting setup from Biome to oxlint while making small TypeScript/test-comment adjustments needed for the new linter.

Changes:

  • Replaces Biome dependency and lint script with oxlint.
  • Removes Biome and VS Code Biome-specific configuration.
  • Refactors hover tooltip data handling and updates one lint-disable directive.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
package.json Switches the lint script and dev dependency from Biome to oxlint.
pnpm-lock.yaml Updates lockfile entries for oxlint and removes Biome packages.
biome.json Removes the previous Biome formatter/linter configuration.
.vscode/settings.json Removes Biome formatter and code-action settings.
src/sql/hover.ts Hoists keyword lookup and preserves narrowed tooltip data for closure use.
src/sql/__tests__/namespace-utils.test.ts Migrates the explicit-any suppression comment to oxlint syntax.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json
"dev": "vite",
"typecheck": "tsc --noEmit",
"lint": "biome check --write",
"lint": "oxlint --fix",
@Light2Dark Light2Dark merged commit 03ffef8 into main May 29, 2026
4 of 5 checks passed
@Light2Dark Light2Dark deleted the sham/move-to-oxlint branch May 29, 2026 14:41
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