Skip to content

chore(FR-2917): remove unused Alert import in VFolderNodes.tsx#7469

Open
ironAiken2 wants to merge 1 commit into
mainfrom
05-18-chore_remove_unused_alert_import_in_vfoldernodes.tsx
Open

chore(FR-2917): remove unused Alert import in VFolderNodes.tsx#7469
ironAiken2 wants to merge 1 commit into
mainfrom
05-18-chore_remove_unused_alert_import_in_vfoldernodes.tsx

Conversation

@ironAiken2
Copy link
Copy Markdown
Contributor

@ironAiken2 ironAiken2 commented May 18, 2026

Resolves #7471(FR-2917)

Summary

Removes an unused Alert import from react/src/components/VFolderNodes.tsx.

Alert was left in the antd import after its last usage was removed (in 0ca1f8b1d, FR-2819 / #7370) but the import line was not cleaned up. It is now dead.

Why split out

This is not part of any feature work. It surfaced because the strict pre-commit hook (eslint ./src --max-warnings=0 via lint-staged) lints the whole react/src tree, so this pre-existing no-unused-vars warning on main blocks unrelated commits. Splitting it into its own one-line PR keeps the FR-26 PR (#7455, stacked on top of this) scope-clean while unblocking the strict hook.

Changes

  • react/src/components/VFolderNodes.tsx — drop Alert from the antd import (no behavior change; the symbol had no remaining references).

Test plan

  • eslint ./src --max-warnings=0 no longer reports the 'Alert' is defined but never used warning.
  • No runtime behavior change (import-only removal).

🤖 Generated with Claude Code

Pre-existing unused import on main that the strict pre-commit hook (eslint ./src --max-warnings=0) flags, blocking unrelated commits. Split out as its own change to keep FR-26 scope clean.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor Author

ironAiken2 commented May 18, 2026


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • flow:merge-queue - adds this PR to the back of the merge queue
  • flow:hotfix - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ironAiken2 ironAiken2 marked this pull request as ready for review May 18, 2026 10:09
@ironAiken2 ironAiken2 requested review from agatha197, Copilot and nowgnuesLee and removed request for Copilot May 18, 2026 10:09
@github-actions
Copy link
Copy Markdown
Contributor

Coverage Report for react-coverage (./react)

Status Category Percentage Covered / Total
🔵 Lines 6.51% 1785 / 27379
🔵 Statements 5.36% 1980 / 36910
🔵 Functions 5.25% 296 / 5634
🔵 Branches 3.74% 1293 / 34569
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
react/src/components/VFolderNodes.tsx 0% 0% 0% 0% 51-691
Generated in workflow #830 for commit 0af3b9a by the Vitest Coverage Report Action

@nowgnuesLee nowgnuesLee changed the title chore: remove unused Alert import in VFolderNodes.tsx chore(FR-2917): remove unused Alert import in VFolderNodes.tsx May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS ~10 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove unused Alert import in VFolderNodes.tsx

1 participant