FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

🐳 Add missing php extensions by LegeBeker · Pull Request #25 · brixion/github-runner · GitHub

/ github-runner Public

🐳 Add missing php extensions - #25

Merged
LegeBeker merged 2 commits into
mainfrom
fix/add-missing-php-extensions
Apr 7, 2026
ConversationCommits (2)ChecksFiles changed
Merged

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

Conversation

LegeBeker commented Apr 7, 2026

Copy link
Copy Markdown
Member

✅ Checklist

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

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

coderabbitai Bot commented Apr 7, 2026 •
edited
Loading

Copy link
Copy Markdown

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

Reviewers

No reviews

Assignees

LegeBeker

Labels

type: 🐛 bug A problem that needs to be fixed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

Back | FazBrowse Home | New Git URL