Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for PHP 7.4 #125

Merged
merged 1 commit into from
Aug 22, 2019
Merged

Conversation

theodorejb
Copy link
Contributor

In PHP 7.4 errors are reported for stream read and write operations per
php/php-src@d59aac5.

Tested with PHP 7.4 beta 4.

@l0gicgate
Copy link
Member

Is this a WIP? If so please label the title with [WIP].

Please fix PHP Stan errors as well

@coveralls
Copy link

coveralls commented Aug 22, 2019

Coverage Status

Coverage increased (+0.003%) to 99.738% when pulling 5cb4871 on theodorejb:support-php-7.4 into 99ad31f on slimphp:master.

@theodorejb
Copy link
Contributor Author

This isn't a WIP anymore. I fixed the PHPStan errors.

@l0gicgate
Copy link
Member

We need to add PHP 7.4 to the travis config also, so we can see if the build passes for that version

@l0gicgate l0gicgate added this to the 0.6 milestone Aug 22, 2019
@theodorejb theodorejb force-pushed the support-php-7.4 branch 2 times, most recently from 915e49f to bd10c4a Compare August 22, 2019 18:19
In PHP 7.4 errors are reported for stream read and write operations per
php/php-src@d59aac5.
@theodorejb
Copy link
Contributor Author

I had to allow failures for PHP 7.4 in Travis. One test fails since phpspec/prophecy#432 hasn't been merged yet.

@l0gicgate
Copy link
Member

I had to allow failures for PHP 7.4 in Travis. One test fails since phpspec/prophecy#432 hasn't been merged yet.

We'll have to make sure we update prophecy as soon as that's merged and remove the allow_failure

@l0gicgate l0gicgate merged commit 0d18263 into slimphp:master Aug 22, 2019
@theodorejb theodorejb deleted the support-php-7.4 branch August 22, 2019 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants