Remove PHP 8 from build matrix to avoid hanging CI jobs #1249

osma · 2021-11-12T11:44:16Z

Reasons for creating this PR

GitHub Actions CI build jobs are getting stuck on PHP 8. This causes unnecessary CI resource usage and delays the completion of build jobs by 6 hours.

Link to relevant issue(s), if any

Related to Tests fail and CI hangs on PHP 8 #1243

Description of the changes in this PR

Remove PHP8 from php_versions in the CI build matrix.

Known problems or uncertainties in this PR

It would be better to get tests to pass on PHP8, but that would take more effort, so as a stopgap measure I'm just disabling them for now.

Checklist

phpUnit tests pass locally with my changes
I have added tests that prove my fix is effective or that my feature works (if not, explain why below)
The PR doesn't introduce unintended code changes (e.g. empty lines or useless reindentation)

This is a CI configuration change, no need for new tests.

…1243

codecov · 2021-11-12T11:44:59Z

Codecov Report

Merging #1249 (3721423) into master (4f33798) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1249   +/-   ##
=========================================
  Coverage     69.26%   69.26%           
  Complexity     1646     1646           
=========================================
  Files            32       32           
  Lines          4041     4041           
=========================================
  Hits           2799     2799           
  Misses         1242     1242

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 4f33798...3721423. Read the comment docs.