Skip to content

Add phpcompatibility-wp coding standards to Gitlab template#315

Merged
danielbachhuber merged 1 commit intowp-cli:masterfrom
Saggre:fix/gitlab-require-coding-standards
Oct 28, 2022
Merged

Add phpcompatibility-wp coding standards to Gitlab template#315
danielbachhuber merged 1 commit intowp-cli:masterfrom
Saggre:fix/gitlab-require-coding-standards

Conversation

@Saggre
Copy link
Copy Markdown
Contributor

@Saggre Saggre commented Oct 19, 2022

Add phpcompatibility-wp coding standards to Gitlab template.

Phpcompatibility-wp is required here:

<rule ref="PHPCompatibilityWP"/>

Phpcompatibility-wp is already used in Travis template:

composer global require wp-coding-standards/wpcs
composer global require phpcompatibility/php-compatibility
composer global require phpcompatibility/phpcompatibility-paragonie
composer global require phpcompatibility/phpcompatibility-wp
phpcs --config-set installed_paths $HOME/.composer/vendor/wp-coding-standards/wpcs,$HOME/.composer/vendor/phpcompatibility/php-compatibility,$HOME/.composer/vendor/phpcompatibility/phpcompatibility-paragonie,$HOME/.composer/vendor/phpcompatibility/phpcompatibility-wp

Phpcompatibility-wp will always include php-compatibility and phpcompatibility-paragonie too:
https://github.com/PHPCompatibility/PHPCompatibilityWP/blob/ed2685318d3af3cd9a1676bea8e3ffc3a972b3b1/composer.json#L26

@Saggre Saggre requested a review from a team as a code owner October 19, 2022 11:47
@danielbachhuber danielbachhuber added this to the 2.1.0 milestone Oct 28, 2022
@danielbachhuber danielbachhuber added the command:scaffold-plugin-tests Related to 'scaffold plugin-tests' command label Oct 28, 2022
Copy link
Copy Markdown
Member

@danielbachhuber danielbachhuber left a comment

Choose a reason for hiding this comment

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

Thanks again, @Saggre !

@danielbachhuber danielbachhuber merged commit 0368f18 into wp-cli:master Oct 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

command:scaffold-plugin-tests Related to 'scaffold plugin-tests' command

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants