Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PHPCS: Update to YoastCS 2.0.0
This updates the dependency in
composer.json
, as well as the PHPCS ruleset for YoastCS 2.0.0.Includes:
composer.lock
for just YoastCS and its dependencies.check-cs
script to allow for PHP >= 5.6.PHP 5.6 is now the minimum PHP version everywhere.
Ref: https://github.com/Yoast/yoastcs/releases/tag/2.0.0
CS: deprecate old-style hook and add new-style hook [1]
This deprecates the
yoast_comment_hacks_notification_roles
filter hook in favour of theYoast\WP\Comment\notification_roles
hook.[This needs a changelog entry]
CS: deprecate old-style hook and add new-style hook [2]
This deprecates the
yoast_ch_admin_footer
action hook in favour of theYoast\WP\Comment\admin_footer
hook.[This needs a changelog entry]
CS: deprecate old-style hook and add new-style hook [3]
This deprecates the
yoast_comment_redirect
filter hook in favour of theYoast\WP\Comment\redirect
hook.[This needs a changelog entry]
CS: use short arrays (everywhere)
Testing
See the test instructions in the Yoast/wpseo-woocommerce#451 PR for guidance on how to test this PR.