Skip to content

GH Actions: don't test against PHPCS 4.x (yet)#1277

Merged
wimg merged 1 commit intodevelopfrom
feature/ghactions-disable-testing-against-phpcs-4.x
Apr 6, 2021
Merged

GH Actions: don't test against PHPCS 4.x (yet)#1277
wimg merged 1 commit intodevelopfrom
feature/ghactions-disable-testing-against-phpcs-4.x

Conversation

@jrfnl
Copy link
Copy Markdown
Member

@jrfnl jrfnl commented Apr 6, 2021

As it looks like PHPCS 4.x is still quite a while away (2022 at the earliest), let's stop testing against PHPCS 4.x for the time being.

This should allow build failure reporting to be more accurate, as currently every PR has a failure on PHPCS 4.x due to a bug in some of the new code in 4.x (fix for this was pulled six months ago and still not merged).

The build against PHPCS 4.x should be re-enabled closer to the PHPCS 4.x release.

Note: PHPCSUtils will continue to test against PHPCS 4.x and will update the provided utilities ahead of time, so with a bit of luck, by the time the build against PHPCS 4.x is re-enabled, the build should largely pass thanks to the compatibility layers in PHPCSUtils.

As it looks like PHPCS 4.x is still quite a while away (2022 at the earliest), let's stop testing against PHPCS 4.x for the time being.

This should allow build failure reporting to be more accurate, as currently every PR has a failure on PHPCS 4.x due to a bug in some of the new code in 4.x (fix for this was pulled six months ago and still not merged).

The build against PHPCS 4.x should be re-enabled closer to the PHPCS 4.x release.

Note: PHPCSUtils will continue to test against PHPCS 4.x and will update the provided utilities ahead of time, so with a bit of luck, by the time the build against PHPCS 4.x is re-enabled, the build should largely pass thanks to the compatibility layers in PHPCSUtils.
@jrfnl jrfnl added Type: chores/QA PR: quick merge PR only contains relatively simple changes PR: ready for review labels Apr 6, 2021
@jrfnl jrfnl added this to the 10.0.0 milestone Apr 6, 2021
@jrfnl jrfnl requested a review from wimg April 6, 2021 14:01
@wimg wimg merged commit 54ca651 into develop Apr 6, 2021
@wimg wimg deleted the feature/ghactions-disable-testing-against-phpcs-4.x branch April 6, 2021 14:31
@jrfnl jrfnl removed PR: quick merge PR only contains relatively simple changes PR: ready for review labels Apr 11, 2021
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