Skip to content

🐳 Add missing php extensions#25

Merged
LegeBeker merged 2 commits intomainfrom
fix/add-missing-php-extensions
Apr 7, 2026
Merged

🐳 Add missing php extensions#25
LegeBeker merged 2 commits intomainfrom
fix/add-missing-php-extensions

Conversation

@LegeBeker
Copy link
Copy Markdown
Member

✅ Checklist

  • Code is lokaal getest
  • Tests zijn toegevoegd/aangepast
  • Documentatie bijgewerkt (indien nodig)

@LegeBeker LegeBeker self-assigned this Apr 7, 2026
@LegeBeker LegeBeker added the type: 🐛 bug A problem that needs to be fixed label Apr 7, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 7, 2026

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

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: 11c90bb9-5279-4502-aa8d-fef8f21cb845

📥 Commits

Reviewing files that changed from the base of the PR and between 1b8105c and 44e4747.

📒 Files selected for processing (1)
  • Dockerfile
🚧 Files skipped from review as they are similar to previous changes (1)
  • Dockerfile

📝 Walkthrough

Walkthrough

The Dockerfile's per-PHP-version installation loop was changed to add php<V>-cli, php<V>-common, php<V>-bcmath, and php<V>-gd; php<V>-zip was reformatted into a continued command; php<V>-opcache is installed in a separate apt-get install attempt that suppresses errors (2>/dev/null || true).

Changes

Cohort / File(s) Summary
Dockerfile PHP Dependencies
Dockerfile
Added php<V>-cli, php<V>-common, php<V>-bcmath, php<V>-gd to the per-version install command; reformatted php<V>-zip into a continued line; added a separate php<V>-opcache install invocation that ignores failures (`2>/dev/null

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is a checklist related to the PR process but does not explain what changes are being made or why, though it is somewhat related to the PR intent. Provide a meaningful description explaining what PHP extensions are being added, why they are needed, and what problems they solve.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise (29 characters), includes a Docker emoji, is written in English, and accurately describes the main change: adding missing PHP extensions to 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
  • Commit unit tests in branch fix/add-missing-php-extensions

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

@LegeBeker LegeBeker merged commit 1455110 into main Apr 7, 2026
4 checks passed
@LegeBeker LegeBeker deleted the fix/add-missing-php-extensions branch April 7, 2026 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: 🐛 bug A problem that needs to be fixed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant