Skip to content

RUM-11859: MIGRATION.MD update#2936

Merged
satween merged 1 commit into
developfrom
tvaleev/feature/RUM-11859
Oct 14, 2025
Merged

RUM-11859: MIGRATION.MD update#2936
satween merged 1 commit into
developfrom
tvaleev/feature/RUM-11859

Conversation

@satween

@satween satween commented Oct 13, 2025

Copy link
Copy Markdown
Contributor

What does this PR do?

In response to a user's request, updating the MIGRATION.md to highlight the significant changes to the SDK v3.

Corresponding changes gonna be made for the documentation repo as well.

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)

@satween
satween requested review from a team as code owners October 13, 2025 15:16
Comment thread MIGRATION.MD Outdated
ambushwork
ambushwork previously approved these changes Oct 13, 2025
Comment thread MIGRATION.MD
Comment thread MIGRATION.MD Outdated
@satween
satween force-pushed the tvaleev/feature/RUM-11859 branch from f65f54f to e6b79c9 Compare October 13, 2025 15:39
@satween
satween force-pushed the tvaleev/feature/RUM-11859 branch from e6b79c9 to 234baf5 Compare October 13, 2025 15:43
@satween
satween requested review from 0xnm and ambushwork October 13, 2025 15:43
@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Oct 13, 2025

Copy link
Copy Markdown

🎯 Code Coverage
Patch Coverage: 100.00%
Total Coverage: 70.82% (-0.02%)

View detailed report

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

@codecov-commenter

codecov-commenter commented Oct 13, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.74%. Comparing base (20de5e0) to head (234baf5).
⚠️ Report is 1351 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2936      +/-   ##
===========================================
- Coverage    70.87%   70.74%   -0.13%     
===========================================
  Files          829      829              
  Lines        30384    30384              
  Branches      5183     5183              
===========================================
- Hits         21534    21494      -40     
- Misses        7378     7409      +31     
- Partials      1472     1481       +9     

see 39 files with indirect coverage changes

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

Comment thread MIGRATION.MD

### Trace product changes

The URL provided in the `useCustomEndpoint` method should be the full endpoint URL

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we could just add a single paragraph saying that all useCustomEndpoint methods changed their behavior instead of repeating it for every feature, but I'm okay with that.

@satween
satween merged commit 18545bb into develop Oct 14, 2025
24 checks passed
@satween
satween deleted the tvaleev/feature/RUM-11859 branch October 14, 2025 09:39
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.

4 participants