GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Fix main's red CI: macOS Composer on PHP 8.4 and the setup-php pin comment by mikolalysenko · Pull Request #1102 · SocketDev/socket-patch

Conversation

@mikolalysenko

@mikolalysenko Mikola Lysenko (mikolalysenko) commented Oct 8, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Collaborator

Problem

Every Composer compatibility run since 2026-10-07 23:51Z fails its macOS composer 2.10.3 / php 8.5 cell in "Setup PHP". The 10 most recent runs all failed there, on main and on PRs (e.g. main run 37712459222):

php: command not found
✗ PHP Could not setup PHP 8.5

On 10-07, Homebrew's PHP formulas were reshuffled when PHP 8.6 was published (shivammathur/homebrew-php). Since then, setup-php 2.37.2 (our pinned version, the latest stable) can't install PHP 8.5 on macOS. The other macOS cells (PHP 8.1 and 8.3) still pass.

Fix

Run the macOS 2.10.3 cell on PHP 8.4, which Composer 2.10.3 supports. Ubuntu and Windows keep the 2.10.3 / 8.5 cell, so PHP 8.5 is still covered. The macOS row in docs/testing/composer-compatibility.md is updated to match.

Validation

The macOS legs no longer run on PRs (#1093), so I dispatched the workflow on this branch: run 37715262908 (result in a follow-up comment).

Also: zizmor ref-version-mismatch on ci.yml

The org's Audit GHA Workflows check now fails any merge group that touches workflows (for example #1050, run 37749840803). It flags ci.yml:1400, shivammathur/setup-php@f3e473d1 # v2: the upstream v2 tag has moved off that commit. f3e473d1 is tag 2.37.2 (verified with the GitHub API), so the comment now says # 2.37.2, matching composer-compatibility.yml. The pin itself is unchanged. #1058 has the same one-line fix, but it is a much larger PR.

🤖 Generated with Claude Code

Note

Low Risk
CI matrix and documentation only; no application or patch logic changes.

Overview
Fixes failing macOS Composer compatibility CI after Homebrew/setup-php stopped providing PHP 8.5 once PHP 8.6 landed (Setup PHP step: php: command not found).

The native-macos matrix cell for Composer 2.10.3 now runs on PHP 8.4 instead of 8.5, with an inline comment documenting the setup-php limitation. Ubuntu and Windows still use 2.10.3 / 8.5, so PHP 8.5 coverage is unchanged overall. The macOS row in docs/testing/composer-compatibility.md is updated to 2.10.3 (8.4).

Reviewed by Cursor Bugbot for commit b4cba52. Configure here.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@mikolalysenko @claude
Run the macOS Composer 2.10.3 cell on PHP 8.4
b4cba52 
Since 2026-10-07 23:51Z every Composer compatibility run has failed
its macOS 2.10.3 / PHP 8.5 cell in "Setup PHP": setup-php 2.37.2
cannot install PHP 8.5 from Homebrew now that PHP 8.6 has shipped
("php: command not found", "Could not setup PHP 8.5"). This keeps
main red on every push.

Run that cell on PHP 8.4, which Composer 2.10.3 supports. Ubuntu and
Windows keep the 2.10.3 / PHP 8.5 cell, so 8.5 is still covered.

Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
@mikolalysenko Mikola Lysenko (mikolalysenko) added the ci-janitor Opened by the CI janitor routine (flakes, redundant tests, CI perf) label Oct 8, 2026
@mikolalysenko

Mikola Lysenko (mikolalysenko) commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator Author

Validation: dispatched run 37715262908 on this branch. All three macOS cells passed: composer 2.10.3 / php 8.4, 2.2.30 / php 8.3 and 1.10.28 / php 8.1.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
This was referenced Oct 8, 2026
Fix UTF-16 requirements.txt silently skipped (#721) #724
Merged
Fix hosted gem pinning a version the lock doesn't resolve (#1055) #1060
Merged
Mikola Lysenko (mikolalysenko) and others added 2 commits October 8, 2026 05:41
@mikolalysenko
Merge remote-tracking branch 'origin/main' into ci/composer-macos-php84
4c8958e
@mikolalysenko @claude
Label ci.yml's setup-php pin with its exact tag
e74ec7d 
The upstream v2 tag moved off f3e473d1, so zizmor's ref-version-mismatch
audit now fails every merge group that touches workflows. f3e473d1 is
2.37.2, the label composer-compatibility.yml already uses.

Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
@mikolalysenko Mikola Lysenko (mikolalysenko) changed the title Run the macOS Composer 2.10.3 cell on PHP 8.4 Fix main's red CI: macOS Composer on PHP 8.4 and the setup-php pin comment Oct 8, 2026
@mikolalysenko
Mikola Lysenko (mikolalysenko) enabled auto-merge October 8, 2026 11:01
@mikolalysenko
Merge branch 'main' into ci/composer-macos-php84
0351101
Wenxin-Jiang
Wenxin Jiang (Wenxin-Jiang) approved these changes Oct 8, 2026
View reviewed changes
@mikolalysenko
Mikola Lysenko (mikolalysenko) disabled auto-merge October 8, 2026 13:04
@mikolalysenko
Mikola Lysenko (mikolalysenko) added this pull request to the merge queue Oct 8, 2026
Merged via the queue into main with commit 33cc8ee Oct 8, 2026
211 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@mikolalysenko
Mikola Lysenko (mikolalysenko) deleted the ci/composer-macos-php84 branch October 8, 2026 13:48
@mikolalysenko Mikola Lysenko (mikolalysenko) mentioned this pull request Oct 8, 2026
CI performance dashboard #1182
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Wenxin-Jiang Wenxin Jiang (Wenxin-Jiang) Wenxin-Jiang approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

ci-janitor Opened by the CI janitor routine (flakes, redundant tests, CI perf)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@mikolalysenko @Wenxin-Jiang
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.