Skip to content

GH Actions: use the xmllint-validate action runner and enhance checks#1798

Merged
wimg merged 3 commits intodevelopfrom
feature/ghactions-change-xmllint-validation
Jul 6, 2025
Merged

GH Actions: use the xmllint-validate action runner and enhance checks#1798
wimg merged 3 commits intodevelopfrom
feature/ghactions-change-xmllint-validation

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Mar 3, 2025

GH Actions: split XML code style check off from "Basic QA"check

The intention is for there to be a dedicated action runner available at some point for XML code style checking, so let's move this to a separate job.

Also see: PHPCSStandards/PHPCSDevTools#145

GH Actions: use the xmllint-validate action runner

Instead of doing all the installation steps for xmllint validation in the workflow, use the ✨ new dedicated phpcsstandards/xmllint-validate action runner instead.

Ref: https://github.com/marketplace/actions/xmllint-validate

GH Actions: add some additional XML validation checks

... for dev tool files.

@jrfnl jrfnl added Type: chores/QA PR: quick merge PR only contains relatively simple changes PR: ready for review labels Mar 3, 2025
@jrfnl jrfnl added this to the 10.0.0 milestone Mar 3, 2025
@jrfnl jrfnl requested a review from wimg March 3, 2025 02:33
jrfnl added 3 commits July 6, 2025 15:32
The intention is for there to be a dedicated action runner available at some point for XML code style checking, so let's move this to a separate job.

Also see: PHPCSStandards/PHPCSDevTools#145
Instead of doing all the installation steps for xmllint validation in the workflow, use the ✨ new dedicated `phpcsstandards/xmllint-validate` action runner instead.

Ref: https://github.com/marketplace/actions/xmllint-validate
@jrfnl jrfnl force-pushed the feature/ghactions-change-xmllint-validation branch from 28e63df to 6335170 Compare July 6, 2025 13:32
@jrfnl
Copy link
Member Author

jrfnl commented Jul 6, 2025

Rebased without changes after the merge of #1806 to allow for CI to pass.

@wimg wimg merged commit 8fc92ce into develop Jul 6, 2025
48 checks passed
@wimg wimg deleted the feature/ghactions-change-xmllint-validation branch July 6, 2025 15:53
@github-actions github-actions bot removed PR: quick merge PR only contains relatively simple changes PR: ready for review labels Jul 6, 2025
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