Skip to content

refactor: reordered PHP tracer installation examples from APM-only to… - #40076

Merged
aminenaim merged 2 commits into
masterfrom
aminenaim/php-install-defaults-first
Sep 21, 2026
Merged

aminenaim merged 2 commits into
masterfrom
aminenaim/php-install-defaults-first

Conversation

@aminenaim

@aminenaim aminenaim commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

Reorders the PHP installer commands so the APM-only install comes first and add a note advising to install only required capabilities

Today the full APM + AAP + Profiling command is listed first, and readers/agents often accidentally copy it by default.

Before:

# Full installation: APM + AAP + Profiling
php datadog-setup.php --php-bin=all --enable-appsec --enable-profiling

# APM only
php datadog-setup.php --php-bin=all

# APM + AAP
php datadog-setup.php --php-bin=all --enable-appsec

# APM + Profiling
php datadog-setup.php --php-bin=all --enable-profiling

After:

# APM only
php datadog-setup.php --php-bin=all

# APM + AAP
php datadog-setup.php --php-bin=all --enable-appsec

# APM + Profiling
php datadog-setup.php --php-bin=all --enable-profiling

# Full installation: APM + AAP + Profiling
php datadog-setup.php --php-bin=all --enable-appsec --enable-profiling

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

Additional notes

@aminenaim
aminenaim requested a review from a team as a code owner September 19, 2026 09:11
@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@aminenaim
aminenaim merged commit b7d3061 into master Sep 21, 2026
24 checks passed
@aminenaim
aminenaim deleted the aminenaim/php-install-defaults-first branch September 21, 2026 14:38
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.

2 participants