Skip to content

Add servers list validation#165

Merged
jovrtn merged 5 commits intoacresources:masterfrom
amoeba:validate-servers
Jul 26, 2025
Merged

Add servers list validation#165
jovrtn merged 5 commits intoacresources:masterfrom
amoeba:validate-servers

Conversation

@amoeba
Copy link
Collaborator

@amoeba amoeba commented Jul 16, 2025 •
edited
Loading

  • Adds an XSD file to validate Servers.xml, XSD has some basic rules
  • Hooks up said XSD to CI so it runs on every PR
  • Updates all entries in Servers.xml to pass validation
  • Does not enforce software names. Should we do this?

@amoeba amoeba requested a review from jovrtn July 16, 2025 01:40
jovrtn
jovrtn approved these changes Jul 26, 2025
View reviewed changes
Copy link
Contributor

@jovrtn jovrtn left a comment

Choose a reason for hiding this comment

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

lgtm

@jovrtn jovrtn merged commit c0859eb into acresources:master Jul 26, 2025
1 check passed
@amoeba
Copy link
Collaborator Author

amoeba commented Jul 26, 2025

🎉

@amoeba amoeba mentioned this pull request Aug 1, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@jovrtn jovrtn jovrtn approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@amoeba @jovrtn