Skip to content

RUM-9894: Replace addFirst usage in PendingTrace#2638

Merged
ambushwork merged 1 commit into
developfrom
yl/RUM-9894/replace-add-first-usage
May 12, 2025
Merged

RUM-9894: Replace addFirst usage in PendingTrace#2638
ambushwork merged 1 commit into
developfrom
yl/RUM-9894/replace-add-first-usage

Conversation

@ambushwork

Copy link
Copy Markdown
Member

What does this PR do?

Replace addFirst usage in PendingTrace, since it can have clash with Android API 35.

Motivation

RUM-9894

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)

@codecov-commenter

codecov-commenter commented May 9, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.94%. Comparing base (f11cf14) to head (9af3ec7).
Report is 156 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2638      +/-   ##
===========================================
- Coverage    70.06%   69.94%   -0.12%     
===========================================
  Files          820      820              
  Lines        30591    30591              
  Branches      5136     5136              
===========================================
- Hits         21432    21396      -36     
- Misses        7730     7747      +17     
- Partials      1429     1448      +19     
Files with missing lines Coverage Δ
...ain/java/com/datadog/opentracing/PendingTrace.java 68.82% <100.00%> (ø)

... and 32 files with indirect coverage changes

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

@ambushwork
ambushwork marked this pull request as ready for review May 12, 2025 11:37
@ambushwork
ambushwork requested review from a team as code owners May 12, 2025 11:37
@ambushwork
ambushwork merged commit af2861f into develop May 12, 2025
@ambushwork
ambushwork deleted the yl/RUM-9894/replace-add-first-usage branch May 12, 2025 13:12
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.

3 participants