Skip to content

Chore: remove dead, commented, and otherwise unused code.#436

Merged
MoralCode merged 6 commits into
mainfrom
chore/cleanup_dead
Jul 2, 2026
Merged

Chore: remove dead, commented, and otherwise unused code.#436
MoralCode merged 6 commits into
mainfrom
chore/cleanup_dead

Conversation

@MoralCode

Copy link
Copy Markdown
Contributor

Description
This PR removes some legacy code that has been left in as comments or un-referenced classes to help make the codebase less confusing to maintain.

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Generative AI disclosure

Please select one option:

  • This contribution was NOT assisted or created by Generative AI tools.
  • This contribution was assisted or created by Generative AI tools.

If AI tools were used, please provide details below:
- What tools were used? Sonnet 4.6 medium
- How were these tools used? broadly checking for obvious cases where code is unused
- Did you review these outputs before submitting this PR? Yes, I verified that the code found was actually unused and used more traditional tools (like vscode's ability to find references) in order to do so.

Signed-off-by: Adrian Edwards <adredwar@redhat.com>
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
Added in 6354aba but the only usage was later removed

Signed-off-by: Adrian Edwards <adredwar@redhat.com>
@MoralCode MoralCode added disclosed-ai ready ready to merge, likely waiting for unit tests or something labels Jul 2, 2026
@MoralCode

Copy link
Copy Markdown
Contributor Author

this has been included in the current batch of PRs being group-tested and seems to not cause any issues (as expected)

@MoralCode MoralCode merged commit 5478d44 into main Jul 2, 2026
16 checks passed
@MoralCode MoralCode deleted the chore/cleanup_dead branch July 2, 2026 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

disclosed-ai ready ready to merge, likely waiting for unit tests or something

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant