Skip to content

Update to shivammathur/[email protected]#741

Merged
simPod merged 2 commits intowebonyx:masterfrom
Kingdutch:patch-1
Nov 27, 2020
Merged

Update to shivammathur/[email protected]#741
simPod merged 2 commits intowebonyx:masterfrom
Kingdutch:patch-1

Conversation

@Kingdutch
Copy link
Copy Markdown
Contributor

@Kingdutch Kingdutch commented Nov 27, 2020

Problem
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 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.

`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.
@Kingdutch
Copy link
Copy Markdown
Contributor Author

The coding Standard, PHPStan and Code Coverage failures are caused by using the master as source for the workflow, thus using the older, broken, version of setup-php. All other tests successfully passed with the 2.9.0 version from this PR.

Copy link
Copy Markdown
Collaborator

@simPod simPod left a 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

@Kingdutch
Copy link
Copy Markdown
Contributor Author

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

@simPod simPod merged commit 8795c03 into webonyx:master Nov 27, 2020
@Kingdutch Kingdutch deleted the patch-1 branch November 27, 2020 11:00

- name: Install PHP
uses: shivammathur/setup-php@2.2.2
uses: shivammathur/setup-php@2.9.0
Copy link
Copy Markdown
Contributor

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.

Copy link
Copy Markdown
Collaborator

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants