Uh oh!
There was an error while loading. Please reload this page.
chore: bump test image to PHP 8.4 - #121
Conversation
composer.json requires `>=8.4` but the test Dockerfile was still pinned at PHP 8.3.3, so any PHP 8.4 syntax (e.g. asymmetric visibility in upgraded dependencies) fails to parse at CI time. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Greptile SummaryThis PR bumps the test Dockerfile's final-stage image from
Confidence Score: 5/5Safe to merge — single-line Dockerfile change swapping the PHP runtime image to a pinned 8.4 tag. The change is a single image tag update with no logic, configuration, or dependency modifications. The new tag pins both the PHP patch version and the Alpine version, maintaining reproducibility. There is nothing here that could introduce incorrect behavior or a regression. No files require special attention. Important Files Changed
Reviews (2): Last reviewed commit: "chore: pin PHP test image to 8.4.18-cli-..." | Re-trigger Greptile |
Uh oh!
There was an error while loading. Please reload this page.
Pinning to a specific patch + Alpine release keeps CI reproducible. Matches the pin used in sibling Appwrite repos. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Summary
composer.jsonrequiresphp: >=8.4(since the migration toutopia-php/query0.1.x), but the testDockerfileis still pinned atphp:8.3.3-cli-alpine3.19. Any PHP 8.4-only syntax in a dependency — e.g. asymmetric visibility (public protected(set)) introduced on the in-flightutopia-php/query0.3.x branch — fails to parse before tests even start.Bumps the image to
php:8.4-cli-alpine. No other changes.Test plan
🤖 Generated with Claude Code