Repository navigation
Stop building PHP 5.6 to 7.3 - #52
Merged
Merged
Conversation
Nothing we run uses these images: every running PHP service and server is on 7.4 or 8.3, and the only one on an older PHP (the stock API, 7.1) was stopped on 22 Oct 2026. Since then the 5.6 to 7.2 builds have also failed at pecl install because pecl.php.net returns bad REST data. Removing them from versions.json stops CI and the Makefile building, testing and publishing them. The published tags stay on Docker Hub, and the README now lists them as no longer built. Fixes #50 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Uugex1w8qZoTbBRbcFwkcp
There was a problem hiding this comment.
🟢 Approval recommended
The narrow configuration and documentation changes match the requested retirement scope, with no unresolved findings.
No open issues 🎉
What changed in this PR
Retires PHP 5.6–7.3 from automated builds, tests, and publishing, addressing #50 while leaving existing Docker Hub tags available.
Changes:
- Removes the five retired versions from the shared list used by CI and aggregate Makefile targets, retaining PHP 7.4 and 8.0–8.5.
- Documents that retired tags remain available but no longer receive updates.
| File | Description |
|---|---|
versions.json |
Removes PHP 5.6–7.3 from the active version list. |
README.md |
Marks retired tags as no longer built or updated. |
Review effort: Balanced
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Fixes #50
Stops CI building, testing and publishing PHP 5.6, 7.0, 7.1, 7.2 and 7.3. Nothing we run uses them, and the 5.6 to 7.2 builds fail on every PR because pecl.php.net returns bad REST data.
Notable Changes
versions.json: 5.6, 7.0, 7.1, 7.2 and 7.3 removed. The workflow and the Makefile both read their version list from this file, so this is the only switch. 7 versions remain (7.4, 8.0 to 8.5), 14 build jobs.README.md: those tags are listed as no longer built. They stay on Docker Hub, so anything that still pulls them keeps working.5.6alias case in the workflow andlatest_5in the Makefile, and graze-docker'snew-php:7.3build.QA
jq -r '.[].version' versions.jsonlists 7.4, 8.0, 8.1, 8.2, 8.3, 8.4, 8.5.Deployment
None.
🤖 Generated with Claude Code
https://claude.ai/code/session_01Uugex1w8qZoTbBRbcFwkcp