Skip to content

Conversation

@dbpolito
Copy link
Member

No description provided.

Copilot finished reviewing on behalf of dbpolito November 12, 2025 20:01
Copy link

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 removes support for PHP 8.0 builds and upgrades the h5bp/server-configs-nginx dependency from version 3.3.0 to 5.0.1 across all remaining nginx-based Docker images.

  • Removes all PHP 8.0 build variants (swoole, swoole-prod, swoole-nginx, swoole-nginx-prod) from the build configuration
  • Updates h5bp/server-configs-nginx dependency from 3.3.0 to 5.0.1 for all nginx-based images
  • Reorganizes documentation to classify PHP 8.0 as a legacy version

Reviewed Changes

Copilot reviewed 22 out of 22 changed files in this pull request and generated no comments.

Show a summary per file
File Description
fwd-template.json Removed all four 8.0 build configurations (8.0-swoole, 8.0-swoole-prod, 8.0-swoole-nginx, 8.0-swoole-nginx-prod)
README.md Reorganized version documentation to move 8.0 to "Legacy Versions" section with consolidated links
template/Dockerfile-nginx.blade.php Updated h5bp/server-configs-nginx version from 3.3.0 to 5.0.1 in the template
8.{1,2,3,4}-swoole-nginx/Dockerfile Updated h5bp/server-configs-nginx version from 3.3.0 to 5.0.1 in all nginx variant Dockerfiles
8.{1,2,3,4}-swoole-nginx-prod/Dockerfile Updated h5bp/server-configs-nginx version from 3.3.0 to 5.0.1 in all nginx production variant Dockerfiles
8.0-swoole*/Dockerfile Deleted all PHP 8.0 Dockerfiles
8.0-swoole-nginx*/supervisor.conf Deleted supervisor configuration files for 8.0 nginx variants
8.0-swoole-nginx*/entrypoint Deleted entrypoint scripts for 8.0 nginx variants
8.0-swoole-nginx*/default.tmpl Deleted nginx configuration templates for 8.0 nginx variants
.github/workflows/ci-cd.yml Removed '8.0-swoole' from the CI/CD build matrix

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dbpolito dbpolito merged commit 8133eef into main Nov 12, 2025
14 checks passed
@dbpolito dbpolito deleted the chore branch November 12, 2025 20:11
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