Skip to content

test(tracing): migrate off backtick operator - #4109

Merged
morrisonlevi merged 1 commit into
masterfrom
levi/backtick
Aug 14, 2026
Merged

test(tracing): migrate off backtick operator#4109
morrisonlevi merged 1 commit into
masterfrom
levi/backtick

Conversation

@morrisonlevi

@morrisonlevi morrisonlevi commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Description

Removes our own usage of the backtick operator because this operator was deprecated in PHP 8.5. This removes deprecation warnings in the logs.

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

This operator was deprecated in PHP 8.5.
@morrisonlevi
morrisonlevi requested a review from a team as a code owner August 14, 2026 16:52
@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Aug 14, 2026

Copy link
Copy Markdown

Pipelines  Tests

Unblock PR with BitsAI

⚠️ Warnings

🚦 42 Pipeline jobs failed

DataDog/apm-reliability/dd-trace-php | ASAN test_c: [8.0, arm64]   View in Datadog   GitLab

DataDog/apm-reliability/dd-trace-php | ASAN test_c: [8.1, arm64]   View in Datadog   GitLab

DataDog/apm-reliability/dd-trace-php | test_extension_ci: [8.1]   View in Datadog   GitLab

View all 42 failed jobs.

1 Test performance regression detected

testUserSignUp from symfony-44-test.DDTrace\Tests\Integrations\Symfony\V4_4\AutomatedLoginEventsTest — 13.9s (+13.05s, +1524%)   View in Datadog

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 8233fcb | Docs | Datadog PR Page | Give us feedback!

@morrisonlevi
morrisonlevi merged commit 22015fe into master Aug 14, 2026
2104 of 2165 checks passed
@morrisonlevi
morrisonlevi deleted the levi/backtick branch August 14, 2026 17:17
@github-actions github-actions Bot added this to the 1.24.0 milestone Aug 14, 2026
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