Skip to content

Travis: fix PHP 8.0 build#461

Merged
schlessera merged 1 commit into
masterfrom
feature/travis-fix-php8
Apr 16, 2021
Merged

Travis: fix PHP 8.0 build#461
schlessera merged 1 commit into
masterfrom
feature/travis-fix-php8

Conversation

@jrfnl
Copy link
Copy Markdown
Member

@jrfnl jrfnl commented Mar 21, 2021

Follow up on #439

Even though we'll soon be moving to GH Actions, I think it is important to fix the failing build on PHP 8.0 ahead of this and ahead of the 1.8.0 release, as otherwise any badges in the README, on Packagist and on the website, will show the CI build as failing until we release version 2.0.0 and that wouldn't be right.

Even though we'll soon be moving to GH Actions, I think it is important to fix the failing build on PHP 8.0 ahead of this and ahead of the 1.8.0 release, as otherwise any badges in the README, on Packagist and on the website, will show the CI build as failing until we release version 2.0.0 and that wouldn't be right.
@jrfnl jrfnl added this to the 1.8.0 milestone Mar 21, 2021
@jrfnl jrfnl requested a review from schlessera March 21, 2021 01:26
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 21, 2021

Codecov Report

Merging #461 (5bc2d5c) into master (18cc33e) will decrease coverage by 0.11%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #461      +/-   ##
============================================
- Coverage     93.06%   92.95%   -0.12%     
  Complexity      764      764              
============================================
  Files            21       21              
  Lines          1788     1788              
============================================
- Hits           1664     1662       -2     
- Misses          124      126       +2     
Impacted Files Coverage Δ Complexity Δ
library/Requests/Transport/fsockopen.php 93.78% <0.00%> (-0.57%) 69.00% <0.00%> (ø%)
library/Requests/Transport/cURL.php 94.23% <0.00%> (-0.49%) 67.00% <0.00%> (ø%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 18cc33e...5bc2d5c. Read the comment docs.

@jrfnl
Copy link
Copy Markdown
Member Author

jrfnl commented Mar 21, 2021

@schlessera schlessera merged commit d42914a into master Apr 16, 2021
@schlessera schlessera deleted the feature/travis-fix-php8 branch April 16, 2021 10:21
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