Upgrade production image to PHP 8.5 - #49
Open
daedeloth wants to merge 1 commit into
Open
Conversation
Move the Docker base image from PHP 8.0 to 8.5 (latest stable) instead of pinning dependencies down to PHP 8.0 compatibility. Only nette/schema and nette/utils capped the platform version in the lock file; updating them lifts the cap, and phpbrake stays on v1.0.0 which requires PHP 8.1+. Adds a regression test asserting the notifier keeps remoteConfig disabled, since phpbrake would otherwise phone home to airbrake.io and silently disable notifications for a self-hosted Errbit. Verified: full docker build succeeds, artisan boots in the image, unit suite green, live smoke test against errors.catlab.eu accepted a notice. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013HqprN6hPwwHLXeCRbGdgo
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
1 participant
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
Instead of pinning phpbrake down to PHP 8.0 (#48), this upgrades production to the latest stable PHP.
thecodingmachine/php:8.0-v5-slim-apache→8.5-v5-slim-apachecomposer.lock: onlynette/schemaandnette/utilscapped the PHP version; targeted update lifts the cap (composer why-not php 8.5is now clean). phpbrake stays on v1.0.0 (requires PHP ≥ 8.1, and itsremoteConfigconstructor bug is fixed there).remoteConfigdisabled, otherwise phpbrake phones home to airbrake.io and silently disables all notifications for a self-hosted Errbit.Verification
docker buildof this branch succeeds (composer platform checks pass inside the container, npm prod build OK)php artisan --versionboots inside the built image (Laravel 9.0.2, deprecation warnings only)Notes
laravel/framework 9.0.* → ^9.52update (last of the 9.x line, ~3 years of fixes) would quiet most of those; kept out of this PR to keep the diff reviewable.🤖 Generated with Claude Code
https://claude.ai/code/session_013HqprN6hPwwHLXeCRbGdgo