Skip to content

chore(gui-test): implement vulture tool to detect dead code and remove those dead code#976

Draft
Asmitapaudel wants to merge 1 commit into
opencloud-eu:mainfrom
Asmitapaudel:chore/vulture-unused-vars
Draft

chore(gui-test): implement vulture tool to detect dead code and remove those dead code#976
Asmitapaudel wants to merge 1 commit into
opencloud-eu:mainfrom
Asmitapaudel:chore/vulture-unused-vars

Conversation

@Asmitapaudel

@Asmitapaudel Asmitapaudel commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Part of #553

Vulture is a static code analyzer that finds unused/dead code in Python projects.

Add vulture to the lint dependency group for dead code detection.

Confidence levels:

  • 100% → definitely unused
  • 80% → pretty sure unused
  • 60% → might be unused (noisy, not recommended)

Using min-confidence = 80 to avoid false positives.

Commands to scan by category:

# unused variables
uv run vulture . | grep "unused variable"

# unused imports
uv run vulture . | grep "unused import"

# unused functions
uv run vulture . | grep "unused function"

# unused classes
uv run vulture . | grep "unused class"

# all at once
uv run vulture .

Since the codebase had minimal dead code after running vulture, all categories (variables, imports, functions, classes) are cleaned up in this single PR instead of splitting across multiple PRs.

@Asmitapaudel
Asmitapaudel force-pushed the chore/vulture-unused-vars branch from ece873e to 8f46c6a Compare July 14, 2026 09:08
@Asmitapaudel Asmitapaudel changed the title chore(gui-test): remove unused variables detected by vulture chore(gui-test): remove dead code detected by vulture Jul 14, 2026
@Asmitapaudel Asmitapaudel changed the title chore(gui-test): remove dead code detected by vulture chore(gui-test): add vulture and remove dead code detected by vulture Jul 14, 2026
@anon-pradip anon-pradip changed the title chore(gui-test): add vulture and remove dead code detected by vulture chore(gui-test): implement vulture tool to detect dead code and remove those dead code Jul 14, 2026
@Asmitapaudel
Asmitapaudel force-pushed the chore/vulture-unused-vars branch 2 times, most recently from 2ebb9f4 to bf65a34 Compare July 14, 2026 11:58
@saw-jan

saw-jan commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

@Asmitapaudel let's keep this as it is for now. we will sort out the test code lint setup and rules first and then we can come to this.

@Asmitapaudel
Asmitapaudel force-pushed the chore/vulture-unused-vars branch from bf65a34 to f2776db Compare July 21, 2026 04:52
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.

3 participants