Skip to content

Ruleset file: add schema tags to the ruleset#1803

Merged
wimg merged 1 commit intodevelopfrom
feature/ruleset-add-schema-url
Jul 6, 2025
Merged

Ruleset file: add schema tags to the ruleset#1803
wimg merged 1 commit intodevelopfrom
feature/ruleset-add-schema-url

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Jun 13, 2025

As of PHPCS 3.2.0, PHPCS offers an XSD schema for rulesets which defines what can be used in the XML ruleset file.

As of a couple of weeks ago, the XSD schema is now available via permalinks.

This commit adds the relevant schema tags to the PHPCS Ruleset file(s).

Refs:

@jrfnl jrfnl added this to the 10.0.0 milestone Jun 13, 2025
@jrfnl jrfnl requested a review from wimg June 13, 2025 06:46
@jrfnl jrfnl added PR: quick merge PR only contains relatively simple changes PR: ready for review labels Jul 6, 2025
As of PHPCS 3.2.0, PHPCS offers an XSD schema for rulesets which defines what can be used in the XML ruleset file.

As of a couple of weeks ago, the XSD schema is now available via permalinks.

This commit adds the relevant schema tags to the PHPCS Ruleset file(s).

Refs:
* squizlabs/PHP_CodeSniffer 1433
* https://github.com/squizlabs/PHP_CodeSniffer/releases/tag/3.2.0
* PHPCSStandards/PHP_CodeSniffer 1094
* https://github.com/PHPCSStandards/schema.phpcodesniffer.com
@jrfnl
Copy link
Member Author

jrfnl commented Jul 6, 2025

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

@jrfnl jrfnl force-pushed the feature/ruleset-add-schema-url branch from 25a0ee4 to 8516270 Compare July 6, 2025 13:33
@wimg wimg merged commit bb3ae51 into develop Jul 6, 2025
46 checks passed
@wimg wimg deleted the feature/ruleset-add-schema-url branch July 6, 2025 15:54
@github-actions github-actions bot removed PR: ready for review PR: quick merge PR only contains relatively simple changes 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.

2 participants