Skip to content

chore: drop unused PHP 8.2 and 8.3 test images - #119

Merged
loks0n merged 1 commit into
mainfrom
chore/php-8.4
Jul 31, 2026
Merged

chore: drop unused PHP 8.2 and 8.3 test images#119
loks0n merged 1 commit into
mainfrom
chore/php-8.4

Conversation

@loks0n

@loks0n loks0n commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

composer.json already requires >=8.4.1, and docker-compose.yml defaults to PHP_VERSION=8.4, so Dockerfile.php-8.2 and Dockerfile.php-8.3 are dead files that no workflow or compose service references. Removing them.

No composer.json change needed here — the minimum is already above 8.4.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@greptile-apps

greptile-apps Bot commented Jul 31, 2026

Copy link
Copy Markdown

Greptile Summary

Removes obsolete Docker test images for unsupported PHP versions.

  • Deletes the PHP 8.2 test-image Dockerfile.
  • Deletes the PHP 8.3 test-image Dockerfile.
  • Retains alignment with the PHP 8.4.1 minimum requirement and the PHP 8.4/8.5 CI matrix.

Confidence Score: 5/5

The PR appears safe to merge because current workflows and supported PHP versions do not rely on either deleted Dockerfile.

The package requires PHP 8.4.1 or newer, CI tests PHP 8.4 and 8.5, and no current repository workflow, script, service, or documentation directly references the removed PHP 8.2 or 8.3 images.

Important Files Changed

Filename Overview
Dockerfile.php-8.2 Deletes an unreferenced test-image definition for PHP 8.2, which is below the package's supported PHP minimum.
Dockerfile.php-8.3 Deletes an unreferenced test-image definition for PHP 8.3, which is below the package's supported PHP minimum.

Reviews (1): Last reviewed commit: "chore: drop unused PHP 8.2 and 8.3 test ..." | Re-trigger Greptile

@loks0n
loks0n merged commit 47d8270 into main Jul 31, 2026
6 checks passed
@loks0n
loks0n deleted the chore/php-8.4 branch July 31, 2026 14:51
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.

1 participant