ci: Reduce required run time

[nickvergessen]

• ci: Run things we only run on 1 PHP version on the preinstalled 8.1 which installs ~2m faster
• ci: Don't run things normally and with coverage flag on the same configuration

[susnux]

looks good to me, hope github images move to 24.04 soon :)

[come-nc]

Not sure whether the include will not add back 8.3?
Also, would prefer 8.1,8.3 over 8.1,8.2.

[come-nc]

I misunderstood how include works, according to the description of the PR 🤔

[susnux]

This change here will result in:

• PHP 8.1 coverage ❌
• PHP 8.2 coverage ❌
• PHP 8.3 coverage ✅

[nickvergessen]

Not sure whether the include will not add back 8.3?

It will and that's exactly the fix. We ran 8.3 (and in other cases 8.2) twice. Once with the coverage flag and once without. But the only difference is whether we generate and upload the report, so we can remove the 8.3 run that is duplicated by the run that also uploads the coverage report

[solracsf]

Question: why don't use ubuntu-20.04 image which has all php versions pre-installed?

https://github.com/shivammathur/setup-php?tab=readme-ov-file#github-hosted-runners

[susnux]

Question: why don't use ubuntu-20.04 image which has all php versions pre-installed?

Because we use self-hosted runners ;)

[st3iny]

🚀

[nickvergessen]

/backport to stable30