Skip to content

Conversation

@cataphract
Copy link
Contributor

Description

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@cataphract cataphract requested a review from a team as a code owner January 20, 2026 11:36
@datadog-official
Copy link

datadog-official bot commented Jan 20, 2026

⚠️ Tests

Fix all issues with Cursor

⚠️ Warnings

🧪 1024 Tests failed

    testSearchPhpBinaries from integration.DDTrace\Tests\Integration\PHPInstallerTest (Fix with Cursor)

testSimplePushAndProcess from laravel-58-test.DDTrace\Tests\Integrations\Laravel\V5_8\QueueTest (Datadog) (Fix with Cursor)
DDTrace\Tests\Integrations\Laravel\V5_8\QueueTest::testSimplePushAndProcess
Test code or tested code printed unexpected output: spanLinksTraceId: 696f617d00000000b00a4095439fbbec
tid: 696f617d00000000
hexProcessTraceId: b00a4095439fbbec
hexProcessSpanId: 5fc846ea47a30f0e
processTraceId: 12685022310271269868
processSpanId: 6901844400983445262

phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:106
testSimplePushAndProcess from laravel-8x-test.DDTrace\Tests\Integrations\Laravel\V8_x\QueueTest (Datadog) (Fix with Cursor)
DDTrace\Tests\Integrations\Laravel\V8_x\QueueTest::testSimplePushAndProcess
Test code or tested code printed unexpected output: spanLinksTraceId: 696f61bf000000002ae0abc52a8391af
tid: 696f61bf00000000
hexProcessTraceId: 2ae0abc52a8391af
hexProcessSpanId: ca9e3b771c9d7bfc
processTraceId: 3089658207686332847
processSpanId: 14600172424748956668
View all

ℹ️ Info

❄️ No new flaky tests detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: ecda295 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@cataphract cataphract merged commit 01cc88c into master Jan 20, 2026
2006 of 2010 checks passed
@cataphract cataphract deleted the glopes/change-helper-conn branch January 20, 2026 12:34
@github-actions github-actions bot added this to the 1.16.0 milestone Jan 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants