Skip to content

fix: stop forcing github runner mode for setup-php - #47

Merged
Soner (shyim) merged 1 commit into
mainfrom
fix/setup-php-self-hosted-runner-env
Jul 28, 2026
Merged

fix: stop forcing github runner mode for setup-php#47
Soner (shyim) merged 1 commit into
mainfrom
fix/setup-php-self-hosted-runner-env

Conversation

@BrocksiNet

Copy link
Copy Markdown
Contributor

Summary

  • Remove the forced RUNNER=github / ImageOS / GITHUB_EMULATION env override from the Setup PHP step
  • After the setup-php 2.37.2 bump in chore: update actions #46, that override conflicts with RUNNER_ENVIRONMENT=self-hosted on runs-on and aborts with Runner set as github in self-hosted environment
  • Let setup-php auto-detect the runner so Rufus/downstream CI can set up again

Test plan

  • Merge and confirm a Rufus Downstream run using setup-shopware@main gets past Setup Rufus / Setup PHP
  • Spot-check a workflow that uses this action on runs-on still installs the requested PHP version and extensions

setup-php 2.37.2 treats RUNNER_ENVIRONMENT=self-hosted as self-hosted
even when ImageOS is set. Forcing RUNNER=github on runs-on then fails
with "Runner set as github in self-hosted environment". Let setup-php
auto-detect instead.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@shyim
Soner (shyim) merged commit 1ae2321 into main Jul 28, 2026
13 checks passed
@shyim
Soner (shyim) deleted the fix/setup-php-self-hosted-runner-env branch July 28, 2026 13:13
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