Skip to content

[php] Fix Laravel11x tests - #7598

Closed
estringana wants to merge 3 commits into
mainfrom
estringana/fix-test-php
Closed

[php] Fix Laravel11x tests#7598
estringana wants to merge 3 commits into
mainfrom
estringana/fix-test-php

Conversation

@estringana

Copy link
Copy Markdown
Contributor

Motivation

Changes

Workflow

  1. ⚠️ Create your PR as draft ⚠️
  2. Work on you PR until the CI passes
  3. Mark it as ready for review
    • Test logic is modified? -> Get a review from RFC owner.
    • Framework is modified, or non obvious usage of it -> get a review from R&P team

🚀 Once your PR is reviewed and the CI green, you can merge it!

🛟 #apm-shared-testing 🛟

Reviewer checklist

  • Anything but tests/ or manifests/ is modified ? I have the approval from R&P team
  • A docker base image is modified?
    • the relevant build-XXX-image label is present
  • A scenario is added, removed or renamed?

@github-actions

Copy link
Copy Markdown
Contributor

CODEOWNERS have been resolved as:

manifests/php.yml                                                       @DataDog/system-tests-reviewers
utils/build/docker/php/weblogs/laravel11x/routes/web.php                @DataDog/system-tests-reviewers

@estringana estringana changed the title Fix Laravel11x tests [php] Fix Laravel11x tests Aug 26, 2026
@datadog-datadog-prod-us1-2

This comment has been minimized.

@estringana estringana closed this Aug 26, 2026
@cbeauchesne
cbeauchesne deleted the estringana/fix-test-php branch August 27, 2026 09:07
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