Skip to content

ci: skip openssl bug79145.phpt on < 8.1#3745

Merged
morrisonlevi merged 1 commit intomasterfrom
levi/openssl-bug79145
Mar 26, 2026
Merged

ci: skip openssl bug79145.phpt on < 8.1#3745
morrisonlevi merged 1 commit intomasterfrom
levi/openssl-bug79145

Conversation

@morrisonlevi
Copy link
Copy Markdown
Collaborator

Description

This test loops many times and it's too slow sometimes causing the CI to be flaky. It was rewritten upstream in PHP 8.1 so we don't need to skip it on 8.1+.

This is a replacement of #3743, which I merged to the wrong base >.<

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

This test loops many times and it's too slow sometimes causing the
CI to be flaky. It was rewritten upstream in PHP 8.1.
@morrisonlevi morrisonlevi requested a review from a team as a code owner March 26, 2026 18:04
@morrisonlevi morrisonlevi merged commit 6ea3b80 into master Mar 26, 2026
1924 of 2120 checks passed
@morrisonlevi morrisonlevi deleted the levi/openssl-bug79145 branch March 26, 2026 18:05
@github-actions github-actions bot added this to the 1.18.0 milestone Mar 26, 2026
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.

2 participants