Skip to content

Remove FQCN usage#3261

Merged
0xnm merged 1 commit into
developfrom
nogorodnikov/remove-fqcn-usage
Mar 16, 2026
Merged

Remove FQCN usage#3261
0xnm merged 1 commit into
developfrom
nogorodnikov/remove-fqcn-usage

Conversation

@0xnm

@0xnm 0xnm commented Mar 16, 2026

Copy link
Copy Markdown
Member

What does this PR do?

A bit of cleanup: we had some places in the code where Full Qualified Class Names were used for nothing.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@0xnm
0xnm requested review from a team as code owners March 16, 2026 10:14
@datadog-prod-us1-6

This comment has been minimized.

Sign in to view
@codecov-commenter

codecov-commenter commented Mar 16, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 42.85714% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.38%. Comparing base (d31130d) to head (271355a).
⚠️ Report is 169 commits behind head on develop.

Files with missing lines Patch % Lines
.../com/datadog/opentelemetry/trace/OtelSpanLink.java 0.00% 2 Missing ⚠️
...id/okhttp/internal/trace/TracingHeaderTypeUtils.kt 50.00% 2 Missing ⚠️
Additional details and impacted files 
@@             Coverage Diff             @@
##           develop    #3261      +/-   ##
===========================================
- Coverage    71.45%   71.38%   -0.07%     
===========================================
  Files          940      940              
  Lines        34774    34774              
  Branches      5894     5894              
===========================================
- Hits         24846    24822      -24     
- Misses        8295     8305      +10     
- Partials      1633     1647      +14
Files with missing lines Coverage Δ
...datadog/android/okhttp/trace/TracingInterceptor.kt 82.09% <100.00%> (ø)
.../com/datadog/opentelemetry/trace/OtelSpanLink.java 0.00% <0.00%> (ø)
...id/okhttp/internal/trace/TracingHeaderTypeUtils.kt 40.00% <50.00%> (ø)

... and 43 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@0xnm
0xnm merged commit 299d0a1 into develop Mar 16, 2026
27 checks passed
@0xnm
0xnm deleted the nogorodnikov/remove-fqcn-usage branch March 16, 2026 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hamorillo hamorillo hamorillo approved these changes
@kikoveiga kikoveiga kikoveiga approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@0xnm @codecov-commenter @hamorillo @kikoveiga