GH Actions: update PHP version for PHAR boxing

php-parallel-lint / PHP-Parallel-Lint

This tool check syntax of PHP files faster than serial check with fancier output.

Other

287 stars 21 forks source link

GH Actions: update PHP version for PHAR boxing #152

Closed jrfnl closed 7 months ago

jrfnl commented 10 months ago

Looks like Box silently dropped support for PHP 8.1. Upgrading the version on which Box is run to 8.2 should fix this. The resulting PHAR files should still work fine on PHP < 8.2.

Ref: https://github.com/box-project/box/releases

Note: this commit will need to be back-ported to the master branch to get passing builds there again too.

jrfnl commented 7 months ago

I'm going to merge this PR as otherwise we'll not be able to test other changes via CI.

jrfnl commented 7 months ago

Darn! I can't due to the branch protection settings and I can't update those. @grogy Can you please approve & merge this ASAP ?

jrfnl commented 7 months ago

Thank you. I don't see info in reference. But it looks ok :)

That's why I mentioned that they "silently" did this (i.e. no mention in the changelog).

For reference: here is the PR in which Box dropped PHP < 8.1 support.

P.S.: have a look at your DMs on Twitter.

grogy commented 7 months ago

Ahh... thank you for the link

To master I put commit: https://github.com/php-parallel-lint/PHP-Parallel-Lint/commits/master

Twitter - ok ok :-)