Skip to content

docs: Add usage examples for install-php-extensions - #6

Merged
leocavalcante merged 1 commit into
mainfrom
docs/examples-and-instructions
Sep 29, 2026
Merged

leocavalcante merged 1 commit into
mainfrom
docs/examples-and-instructions

Conversation

@leocavalcante

Copy link
Copy Markdown
Member

Summary

  • The repository README and the install-php-extensions feature README now show the feature id ghcr.io/opencodeco/devcontainers/install-php-extensions and the extensions option.
  • Official PHP images, such as php:8.3-cli, run mlocati/docker-php-extension-installer 2.2.5. Debian and Ubuntu images, such as mcr.microsoft.com/devcontainers/base:bullseye, install php-cli and the distro package with apt. zip installs php-zip.

Test plan

  • README or feature docs match devcontainer-feature.json and install.sh

Fixes #1

The repository README had no usage example, and the feature README only showed a Debian base.
@leocavalcante leocavalcante added the documentation Improvements or additions to documentation label Sep 29, 2026
@leocavalcante leocavalcante self-assigned this Sep 29, 2026
@leocavalcante
leocavalcante requested a balanced review from Copilot September 29, 2026 19:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The examples match the feature metadata and implementation, and host-side checks pass.

Review effort: Balanced
Findings: None

What changed in this PR

Adds usage guidance and examples for the PHP extension feature, addressing issue #1.

Changes:

  • Documents supported image types and extension behavior.
  • Adds examples for official PHP and Debian/Ubuntu images.
File Description
README.md Adds feature overview and examples.
features/​install-php-extensions/​README.md Expands usage examples for both installation paths.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@leocavalcante
leocavalcante merged commit 84ca8ad into main Sep 29, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Requires Examples or Instructions

2 participants