Fix #75: phpcompatibility/php-compatibility dev-develop #76
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.
Motivation
Fixes #75
Proposed changes
Update phpcompatibility/php-compatibility to dev-develop branch, as recommended by its maintainers (can switch back to stable after a 10.0 release)
Rohit was right, phpcompatibility 9.x does basically nothing when running in a PHP 8 environment. Case in point: Acquia CLI, which uses a ton of PHP 8 syntax, never saw a violation from this standard on 9.x. Updating to the develop branch revealed thousands of violations.
We should consider this a breaking change. If we release it, projects that don't want to support PHP 7 will need to change the minimum version in their phpcs.xml:
Projects that do support PHP 7 may still expect to see some violations.
Alternatives considered
Testing steps
Try this on projects of various PHP versions