Skip to content

fix: clean up final 16 no-unused-vars warnings#22751

Merged
kitlangton merged 1 commit intodevfrom
kit/oxlint-unused-vars-2
Apr 16, 2026
Merged

fix: clean up final 16 no-unused-vars warnings#22751
kitlangton merged 1 commit intodevfrom
kit/oxlint-unused-vars-2

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

Summary

  • Remove 3 unused imports (CompactionPart, createMemo, en)
  • Delete 1 unused function (prepareBinDirectory in postinstall)
  • Delete 4 dead variables (gate, sessionCount, wrapper, reply)
  • Delete 2 unused Deferred declarations in tests
  • Prefix 4 intentional placeholders with _ (permissionCalls, ghostty, githubData, input)

This should bring no-unused-vars warnings to zero.

Remove unused imports, delete dead code, and prefix intentional
placeholders with underscore. Deletes unused prepareBinDirectory
function and unused Deferred variables in tests.
@kitlangton kitlangton force-pushed the kit/oxlint-unused-vars-2 branch from 9564c18 to 73e7029 Compare April 16, 2026 03:45
@kitlangton kitlangton merged commit 9f4b73b into dev Apr 16, 2026
11 checks passed
@kitlangton kitlangton deleted the kit/oxlint-unused-vars-2 branch April 16, 2026 03:54
jerome-benoit pushed a commit to jerome-benoit/opencode that referenced this pull request Apr 16, 2026
xywsxp pushed a commit to xywsxp/opencode that referenced this pull request Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant