Repository navigation
chore(php): update setup-php to the latest 8.3 patch in CI [skip changelog] - #127
Conversation
…gelog] setup-php reuses the PHP build preinstalled on the runner image when it satisfies the requested minor version, so CI stayed on whatever 8.3.x the image was built with. Setting update: true installs the latest 8.3 patch release on every run.
There was a problem hiding this comment.
No issues found across 1 file
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Shadow auto-approve: would auto-approve. Sets update: true on the setup-php step in the PHP CI workflow so each run installs the latest PHP 8.3 patch, ensuring security fixes are tested promptly instead of waiting for a runner image rebuild. This is a bounded one-line change to development-only CI configuration.
View guided diff | Turn on auto-fix | Re-trigger cubic
setup-php reads update from the environment, not from its inputs.
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Shadow auto-approve: would auto-approve. Moves the setup-php update: true flag into an env var in the PHP CI workflow so each run installs the latest PHP 8.3 patch, since setup-php reads update from the environment. This is a bounded, development-only CI configuration change with no runtime or product impact.
View guided diff | Turn on auto-fix | Re-trigger cubic
bflad
left a comment
There was a problem hiding this comment.
Weird way to do it but confirmed in their docs 👍
Why
The PHP test workflow runs setup-php with
php-version: "8.3". When the requested minor version is preinstalled on the runner image, setup-php switches to that build instead of installing the newest patch, so CI kept running whatever 8.3.x the runner image was built with until the image was rebuilt. PHP security releases should reach CI on the next run, not on the next image refresh.What changed
.github/workflows/test-php.yml: setupdate: trueon the setup-php step so it installs the latest 8.3 patch release on every run.No generator or template changes; no generated output changes.
Testing
git diff --checkis clean.updateis set.Public-safety check
.claude/skills/public-repo-communication/SKILL.md).git diff --check.Summary by cubic
Ensures CI installs the latest PHP 8.3 patch release every run by setting
update: trueon the setup-php step, so security fixes reach CI immediately instead of waiting for a runner image rebuild. The flag is passed as an environment variable since setup-php readsupdatefrom the environment rather than its inputs.Written for commit 16b339f. Summary will update on new commits.