Skip to content

Conversation

@MoralCode
Copy link
Contributor

Description
Simple change to remove some unused files. Separated out from #3435 to make the diff less crazy to review

Signed commits

  • Yes, I signed my commits.

@MoralCode MoralCode added tech debt ready Items tested and seeking additional approvals or a merge. Usually for items under active development labels Dec 3, 2025
@MoralCode
Copy link
Contributor Author

IMO since these files are entirely comments, Im going to consider this change to be tested

Copy link
Member

@sgoggins sgoggins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@sgoggins
Copy link
Member

sgoggins commented Dec 3, 2025

@MoralCode : I am rerunning the podman end to end test. Not sure why its failing here.

@MoralCode
Copy link
Contributor Author

Seems like it passed second time around

Copy link
Collaborator

@shlokgilda shlokgilda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sgoggins sgoggins self-assigned this Dec 6, 2025
@sgoggins sgoggins merged commit 134facd into main Dec 9, 2025
18 of 19 checks passed
@MoralCode MoralCode deleted the ntdn/old_schemas branch December 9, 2025 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready Items tested and seeking additional approvals or a merge. Usually for items under active development tech debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants