Update to shivammathur/[email protected]#741
Conversation
`set-env` and `add-path` are disabled in GitHub actions. See https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/ This requires an update to the action using those commands. This means at least version 2.6.0 is needed. See shivammathur/setup-php#300 However there seem to be no breaking changes so we can go straight to 2.9.0 instead. This also allows testing with version 8.x of PHP.
|
The coding Standard, PHPStan and Code Coverage failures are caused by using the |
|
I had totally missed there were more jobs, fixed it now. Couldn't amend the previous commit locally so had to edit through the GitHub UI. Please squash this during merge :D |
|
|
||
| - name: Install PHP | ||
| uses: shivammathur/setup-php@2.2.2 | ||
| uses: shivammathur/setup-php@2.9.0 |
There was a problem hiding this comment.
FTR, ever since I've been using setup-php it's been extremely stable and the author is very diligent. Aka never have I experienced issues by just using setup-php@2 and benefit from minor updates on the fly.
There was a problem hiding this comment.
Yup but 💩 happens, I guess it's better to lock to minor at least to have reproducible builds.
Problem
set-envandadd-pathare disabled in GitHub actions.See https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/
This causes the tests in all PRs to fail until resolved.
Solution
This requires an update to the action using those
commands. This means at least version 2.6.0 is
needed. See shivammathur/setup-php#300
However there seem to be no breaking changes so we
can go straight to 2.9.0 instead. This also allows testing
with version 8.x of PHP.