Skip to content

Travis: don't allow PHP 7.4 build to fail#935

Merged
wimg merged 1 commit intodevelopfrom
feature/travis-php-74-not-allowed-to-fail
Dec 2, 2019
Merged

Travis: don't allow PHP 7.4 build to fail#935
wimg merged 1 commit intodevelopfrom
feature/travis-php-74-not-allowed-to-fail

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Dec 1, 2019

As PHP 7.4 has been released, the build against PHP 7.4 should no longer be allowed to fail.

Includes some tweaking of the matrix to have the coverage stage and the quick test stage include PHP 7.4.

@jrfnl jrfnl added Type: chores/QA PR: quick merge PR only contains relatively simple changes PR: ready for review labels Dec 1, 2019
@jrfnl jrfnl added this to the 9.x Next milestone Dec 1, 2019
@jrfnl jrfnl requested a review from wimg December 1, 2019 14:49
@jrfnl jrfnl force-pushed the feature/travis-php-74-not-allowed-to-fail branch 2 times, most recently from a63a4c3 to c17936c Compare December 1, 2019 15:01
Copy link
Member

@wimg wimg left a comment

Choose a reason for hiding this comment

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

Looks like Travis isn't ready for this after all ?

As [PHP 7.4 has been released](https://www.php.net/archive/2019.php#2019-11-28-1), the build against PHP 7.4 should no longer be allowed to fail.

Includes some tweaking of the matrix to have the coverage stage and the quick test stage include PHP 7.4.
@jrfnl jrfnl force-pushed the feature/travis-php-74-not-allowed-to-fail branch from c17936c to 73f8953 Compare December 1, 2019 21:25
@jrfnl
Copy link
Member Author

jrfnl commented Dec 1, 2019

Looks like Travis isn't ready for this after all ?

Yes and no. There still isn't a 7.4 image available. However, the 7.4snapshot image should work fine. My bad for mixing them up. Should be fixed now.

@wimg wimg merged commit 62bd429 into develop Dec 2, 2019
@wimg wimg deleted the feature/travis-php-74-not-allowed-to-fail branch December 2, 2019 02:03
@jrfnl jrfnl removed PR: quick merge PR only contains relatively simple changes PR: ready for review labels Dec 22, 2019
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