Update test suite to future-proof base images by clue · Pull Request #299 · reactphp/socket · GitHub
Skip to content

Update test suite to future-proof base images#299

Merged
WyriHaximus merged 1 commit into
reactphp:1.xfrom
clue-labs:runs-on
Oct 15, 2022
Merged

Update test suite to future-proof base images#299
WyriHaximus merged 1 commit into
reactphp:1.xfrom
clue-labs:runs-on

Conversation

@clue

@clue clue commented Oct 6, 2022

Copy link
Copy Markdown
Member

This simple changeset updates the test suite to future-proof base images. Likewise, legacy HHVM is now executed in an up-to-date base image, but using a legacy container instead. This way, we no longer depend on any legacy base images that will be removed in the future as per https://github.blog/changelog/2022-08-09-github-actions-ubuntu-22-04-is-now-generally-available-on-github-hosted-runners/, https://github.blog/changelog/2022-08-09-github-actions-the-ubuntu-18-04-actions-runner-image-is-being-deprecated-and-will-be-removed-by-12-1-22/ and actions/runner-images#6002. I've confirmed pulling container images from Docker Hub does not appear to be rate-limited as per actions/runner-images#1445. Once these changes are merged, we should apply similar updates to all our other components.

Builds on top of #297 and #289

@clue clue added this to the v1.13.0 milestone Oct 6, 2022
@clue clue requested a review from WyriHaximus October 6, 2022 13:13
Comment thread .github/workflows/ci.yml Outdated
@clue

clue commented Oct 11, 2022

Copy link
Copy Markdown
Member Author

@clue

clue commented Oct 12, 2022

Copy link
Copy Markdown
Member Author

@WyriHaximus WyriHaximus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the updates, looks even better now 👍 .

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants