Skip to content

ref(lint): Enable no-unused-vars - #2122

Draft
JoshuaKGoldberg wants to merge 1 commit into
mainfrom
josh/lint-no-unused-vars
Draft

JoshuaKGoldberg wants to merge 1 commit into
mainfrom
josh/lint-no-unused-vars

Conversation

@JoshuaKGoldberg

Copy link
Copy Markdown
Member

Turns on oxlint's no-unused-vars, which went missing when Biome's noUnusedImports was dropped in the oxlint migration, and removes the unused code it flags. Parameters that must stay for a signature use a _ prefix.

🤖 Generated with Claude Code

Turn on oxlint's no-unused-vars (it was disabled in the Biome migration)
and remove the unused imports, variables, parameters, and helpers it
flags.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
sentry-local Ready Ready Preview Nov 9, 2026 9:43pm UTC

Request Review

@JoshuaKGoldberg

Copy link
Copy Markdown
Member Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 26b307a. Configure here.

This branch was successfully deployed

2 active deployments
Preview — 26b307ac Deployed Nov 9, 2026 by vercel[bot]
Actions — 26b307ac Deployed Nov 9, 2026 by JoshuaKGoldberg via eval #1753
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.

1 participant