New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to shivammathur/setup-php@2.9.0 #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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The version needs to be updated for each job
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 |
@@ -24,7 +24,7 @@ jobs: | |||
- uses: actions/checkout@v2 | |||
|
|||
- name: Install PHP | |||
uses: shivammathur/setup-php@2.2.2 | |||
uses: shivammathur/setup-php@2.9.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup but 💩 happens, I guess it's better to lock to minor at least to have reproducible builds.
Problem
set-env
andadd-path
are 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.