Skip to content

Update PHP patch + exts#61

Merged
abnegate merged 1 commit intomainfrom
chore-updates
Mar 4, 2026
Merged

Update PHP patch + exts#61
abnegate merged 1 commit intomainfrom
chore-updates

Conversation

@abnegate
Copy link
Member

@abnegate abnegate commented Mar 4, 2026

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Summary by CodeRabbit

  • Chores
    • Updated PHP base image to version 8.4.18
    • Updated Swoole extension to v6.1.7
    • Updated MongoDB driver to version 2.2.1
    • Updated XDebug to version 3.5.1
    • Updated Protobuf to version 5.34.0

@coderabbitai
Copy link

coderabbitai bot commented Mar 4, 2026

Walkthrough

This change updates multiple dependency versions in a Dockerfile. The PHP base image is updated from version 8.4.17 to 8.4.18, the Swoole extension from v6.1.6 to v6.1.7, the MongoDB driver from 2.1.7 to 2.2.1, XDebug from 3.5.0 to 3.5.1, and Protobuf from 4.33.5 to 5.34.0. All changes are version number updates with no modifications to configuration or build logic.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Update PHP patch + exts' directly relates to the main changeset, which updates the PHP base image patch version and multiple extension versions in the Dockerfile.
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.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore-updates

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@Dockerfile`:
- Line 18: The Dockerfile bump of PHP_PROTOBUF_VERSION to "5.34.0" introduces
breaking API and PHP-version changes that must be communicated and validated:
update release notes/changelog and the project README to call out the Protobuf
major-version bump and removed APIs (e.g., FieldDescriptor::getLabel,
Field_Kind, Field_Cardinality, Internal\RepeatedField) and the PHP >= 8.2
requirement, add functional tests or example usage exercising protobuf
serialization/deserialization in the test matrix (extend tests.yaml beyond
presence checks to run a small proto encode/decode roundtrip using the image),
and scan known downstream consumer repos/dependencies for use of the removed
symbols to decide whether to provide migration guidance or pin older protobuf
versions for compatibility; use the PHP_PROTOBUF_VERSION token in the Dockerfile
as the change anchor when updating docs and CI.

ℹ️ Review info
Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 862af8c2-5fe2-4e67-8cf6-103f1def4d5d

📥 Commits

Reviewing files that changed from the base of the PR and between 54a5d49 and 8cc320a.

📒 Files selected for processing (1)
  • Dockerfile

@abnegate abnegate merged commit 7700869 into main Mar 4, 2026
6 checks passed
@abnegate abnegate deleted the chore-updates branch March 4, 2026 04:16
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