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

Fix PHP 7.4 support for 1.x #206

Merged
merged 1 commit into from
Dec 30, 2019
Merged

Conversation

mlocati
Copy link
Contributor

@mlocati mlocati commented Jul 31, 2019

Backport #203 to 1.x

@mlocati mlocati mentioned this pull request Jul 31, 2019
@egulias
Copy link
Owner

egulias commented Jul 31, 2019

Is this a Travis issue or a problem with Ubuntu´s version?

@mlocati
Copy link
Contributor Author

mlocati commented Jul 31, 2019

I haven't inspected what's going wrong with the failing travisci jobs.
Maybe it was a transient problem? Could you try to restart the jobs?

@egulias
Copy link
Owner

egulias commented Jul 31, 2019

Yes, did it. Same error, a missing global var. Will take a look later.

@mlocati
Copy link
Contributor Author

mlocati commented Aug 1, 2019

What about #207 ?

@egulias
Copy link
Owner

egulias commented Aug 3, 2019

It passes. A substantial config change was need as I see. I´ll take a look to scrutinizer. Should I close this one and we continue in #207 ?

@mlocati
Copy link
Contributor Author

mlocati commented Aug 3, 2019

IMHO #206 and #207 fix two separate issues.
I'd merge #207 and rebase #206 so that tests for php 7.4 passes

@egulias
Copy link
Owner

egulias commented Aug 8, 2019

Merged #207 so you now can update.
Thanks for patience!

@mlocati mlocati force-pushed the fix-php7.4-support-1.x branch from b46675f to 6c9f1a0 Compare August 8, 2019 06:45
@mlocati
Copy link
Contributor Author

mlocati commented Aug 8, 2019

PR rebased. BTW it seems that TravisCI is still having connectivity issues.

@egulias could you try to restart the failing TravisCI jobs?

@egulias egulias merged commit 92fd7ea into egulias:1.2 Dec 30, 2019
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