Skip to content

Bump frankenphp#4375

Merged
ildyria merged 2 commits into
masterfrom
bump-krb
May 26, 2026
Merged

Bump frankenphp#4375
ildyria merged 2 commits into
masterfrom
bump-krb

Conversation

@ildyria
Copy link
Copy Markdown
Member

@ildyria ildyria commented May 26, 2026

Summary by CodeRabbit

  • Chores
    • Updated production runtime to PHP 8.5.
    • Trimmed and optimized system dependencies for a leaner production image.

Review Change Stack

@ildyria ildyria requested a review from a team as a code owner May 26, 2026 16:09
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ed20b93d-f90a-44e6-88f7-75a1b04a4bf5

📥 Commits

Reviewing files that changed from the base of the PR and between 00a029c and eb26395.

📒 Files selected for processing (1)
  • Dockerfile

📝 Walkthrough

Walkthrough

Production Dockerfile: the FrankenPHP base image digest is updated and the production-stage apt package list is modified (several libraries removed; libgssapi-krb5-2 added).

Changes

Container Image Optimization

Layer / File(s) Summary
Base image digest and apt package list
Dockerfile
FROM dunglas/frankenphp:php8.5-trixie is updated to a new digest. The production-stage apt-get install list removes libcap2*, libnghttp2-14, libsystemd0, libudev1, libgnutls30t64 and includes libgssapi-krb5-2.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I hopped through a Dockerfile light,
Swapped the digest and trimmed some bytes,
Removed a few libs, added one more,
A tidy image to explore,
Hops and builds — behold the sight!

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ildyria ildyria merged commit 70ac9ac into master May 26, 2026
46 checks passed
@ildyria ildyria deleted the bump-krb branch May 26, 2026 19:10
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