Skip to content

Clean CodeQL quality findings - #9

Merged
Ryson-32 merged 1 commit into
mainfrom
ryan/06-17-clean-codeql-quality
Jun 17, 2026
Merged

Clean CodeQL quality findings#9
Ryson-32 merged 1 commit into
mainfrom
ryan/06-17-clean-codeql-quality

Conversation

@Ryson-32

Copy link
Copy Markdown
Member

Summary

  • remove unused imports and local variables reported by CodeQL
  • rename repeated test hook/result snapshots to avoid useless-assignment findings
  • keep runtime behavior unchanged; no release, tag, or deploy

Validation

  • git diff --check
  • npm run test:unit -- packages/ui/src/product/converter/advanced-mode/sections/proxy-groups-rules-search.test.ts packages/ui/src/product/converter/advanced-mode/sections/node-management/bulk-edit-dialog.test.ts packages/ui/src/product/converter/advanced-mode/sections/proxy-groups-module-rules-panel.test.ts
  • npm run local:typecheck
  • npm run lint
  • npm run local:build
  • npm run test:unit

@Ryson-32
Ryson-32 merged commit 5ce90e1 into main Jun 17, 2026
5 checks passed
@Ryson-32
Ryson-32 deleted the ryan/06-17-clean-codeql-quality branch June 17, 2026 15:23
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