Skip to content

Conversation

@KingPin
Copy link
Owner

@KingPin KingPin commented Oct 20, 2025

  • Remove PHP 7.x, 8.0 and 8.1 from active CI build/publish matrices and update documentation to mark them as deprecated.
  • Keep previously published images available in registries (Docker Hub / GHCR / Quay) for backwards compatibility; this PR only stops active building/publishing of those versions.
  • Branch: deprecate-php-7-8.1

KingPin added 3 commits October 20, 2025 15:07
- Remove PHP 7 and 8.1 from docker-ci.yml build-and-test matrix
- Remove PHP 7 and 8.1 from docker-ci.yml publish matrix
- Remove PHP 7 and 8.1 from docker-image.v1.yml matrix
- Remove bullseye base OS (only needed for PHP 7)
- Simplify matrix exclusions (now only apache+alpine excluded)
- Add deprecation notice to README with migration path
- Create docs/deprecated-images.md with full deprecation details
- Update docs/migration.md to reference deprecated versions
- Keep PHP 8.2 and 8.3 as actively built versions

Existing images for PHP 7.x and 8.1 remain available in registries
for backwards compatibility but will receive no further updates.
State that PHP 7, 8 and 8.1 builds are no longer published and update the features list; remove the explicit "as of January 2025" date.
… tags table

- Remove 8.1 rows from the "Available Tags" table (8.1 is deprecated)
- Rewrite "Deprecated Tags" section to list PHP 7.x, 8.0, and 8.1 and keep registry availability note
- Update upgrade guidance to recommend PHP 8.2 or 8.3
@KingPin KingPin requested a review from Copilot October 20, 2025 20:39
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR deprecates PHP 7.x, 8.0, and 8.1 versions by removing them from active CI build matrices while keeping previously published images available in container registries. The changes focus on streamlining maintenance to PHP 8.2 and 8.3.

Key changes:

  • CI workflows updated to remove deprecated PHP versions from build/publish matrices
  • Comprehensive documentation added explaining deprecation status and migration paths
  • README updated with deprecation notices and corrected version references

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
docs/migration.md Added note directing users of deprecated versions to new migration guide
docs/deprecated-images.md New comprehensive guide documenting deprecated versions, migration paths, and registry availability
README.md Updated to reflect actively supported versions and expanded deprecated versions section
.github/workflows/docker-image.v1.yml Removed deprecated PHP versions and bullseye from build matrix
.github/workflows/docker-ci.yml Removed deprecated PHP versions and bullseye from CI and publish matrices

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.


- **Multi-Architecture Support**: Works on `amd64`, `arm64/aarch64` and `arm32v7/armhf` platforms
- **Multiple PHP Versions**: PHP 7.x (deprecated), 8.1, 8.2, and 8.3
- **Multiple PHP Versions**: PHP 8.2 and 8.3 (actively built); PHP 7, 8 and 8.1 deprecated
Copy link

Copilot AI Oct 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inconsistent version notation. Should be 'PHP 7.x, 8.0 and 8.1' to match other documentation and the PR title.

Copilot uses AI. Check for mistakes.
KingPin and others added 2 commits October 20, 2025 16:40
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@KingPin KingPin merged commit a49d630 into main Oct 20, 2025
0 of 33 checks passed
@KingPin KingPin deleted the deprecate-php-7-8.1 branch October 20, 2025 20:42
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.

2 participants