Skip to content

Remove unused ExpandRowRecursive after #305#306

Merged
erikdarlingdata merged 1 commit intodevfrom
cleanup/remove-unused-expandrowrecursive
May 3, 2026
Merged

Remove unused ExpandRowRecursive after #305#306
erikdarlingdata merged 1 commit intodevfrom
cleanup/remove-unused-expandrowrecursive

Conversation

@erikdarlingdata
Copy link
Copy Markdown
Owner

Summary

  • PR switch the default grid GroupBy from "None" to "Query Hash" and don't… #305 removed the only caller of ExpandRowRecursive in FetchGroupedPlansAsync. Only its self-recursion remained, making the whole method dead code.
  • Also drop two unused using directives in the same file (Avalonia — Avalonia members are accessed fully qualified everywhere else; Microsoft.Data.SqlClient — no references).

Test plan

  • dotnet build src/PlanViewer.App/PlanViewer.App.csproj -c Debug — 0 errors, only pre-existing XAML loader warnings.

🤖 Generated with Claude Code

PR #305 dropped the only caller of ExpandRowRecursive in
FetchGroupedPlansAsync. Delete the now-unreachable method (only
its self-recursion remained) plus stale Avalonia/SqlClient usings
in the same file.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit 5f588fc into dev May 3, 2026
2 checks passed
@erikdarlingdata erikdarlingdata deleted the cleanup/remove-unused-expandrowrecursive branch May 3, 2026 13:21
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