Skip to content

GH Actions: stop running tests against PHP 5.5/Composer 1.x on Windows (and remove work-arounds)#183

Merged
jrfnl merged 2 commits intomasterfrom
feature/ghactions-remove-problem-build
Jun 25, 2022
Merged

GH Actions: stop running tests against PHP 5.5/Composer 1.x on Windows (and remove work-arounds)#183
jrfnl merged 2 commits intomasterfrom
feature/ghactions-remove-problem-build

Conversation

@jrfnl
Copy link
Copy Markdown
Member

@jrfnl jrfnl commented Jun 24, 2022

Proposed Changes

GH Actions: stop running tests against PHP 5.5/Composer 1.x on Windows

As discussed in #181.

Fixes #181

Tests: remove the TestCase::willPluginOutputShow() helper method and all uses

This method was put in place in PR #153 (commit d4f64b8) as a work-around for the problematic PHP 5.5 / Composer v1 on Windows build.

As we won't be running the tests against that particular combination anymore, I propose we remove all these work-arounds from the test suite.

This effectively reverts the following commits (and a little more):

Related Issues

#181

jrfnl added 2 commits June 24, 2022 15:49
…d all uses

This method was put in place in PR 153 (commit d4f64b8) as a work-around for the problematic PHP 5.5 / Composer v1 on Windows build.

As we won't be running the tests against that particular combination anymore, I propose we remove all these work-arounds from the test suite.

This effectively reverts the following commits (and a little more):
* f0251b8.
* ccd127f.
* d4f64b8.
@jrfnl jrfnl requested a review from Potherca June 24, 2022 14:37
@jrfnl jrfnl changed the title Feature/ghactions remove problem build GH Actions: stop running tests against PHP 5.5/Composer 1.x on Windows (and remove work-arounds) Jun 24, 2022
@jrfnl
Copy link
Copy Markdown
Member Author

jrfnl commented Jun 25, 2022

FYI: I've removed the build I'm now excluding from the "required statuses" in the branch protection rules.

@jrfnl jrfnl merged commit edde434 into master Jun 25, 2022
@jrfnl jrfnl deleted the feature/ghactions-remove-problem-build branch June 25, 2022 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

CI: exclude Windows / Composer v1 / PHP 5.5 from builds

2 participants