Skip to content

RUM-13071: Stop using process importance when distinguishing between cold and warm launches#3030

Merged
aleksandr-gringauz merged 1 commit into
developfrom
aleksandr-gringauz/RUM-13071/stop-using-process-importance-app-launch
Nov 27, 2025
Merged

RUM-13071: Stop using process importance when distinguishing between cold and warm launches#3030
aleksandr-gringauz merged 1 commit into
developfrom
aleksandr-gringauz/RUM-13071/stop-using-process-importance-app-launch

Conversation

@aleksandr-gringauz

@aleksandr-gringauz aleksandr-gringauz commented Nov 27, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

What does this PR do?

See the description in the linked ticket (RUM-13071).

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

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)

@datadog-official

datadog-official Bot commented Nov 27, 2025 •
edited
Loading

Copy link
Copy Markdown

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

View detailed report

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

@codecov-commenter

codecov-commenter commented Nov 27, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.17%. Comparing base (9794bcb) to head (718e2bf).

Additional details and impacted files 
@@             Coverage Diff             @@
##           develop    #3030      +/-   ##
===========================================
+ Coverage    71.16%   71.17%   +0.01%     
===========================================
  Files          859      859              
  Lines        31457    31453       -4     
  Branches      5306     5305       -1     
===========================================
- Hits         22386    22385       -1     
+ Misses        7565     7558       -7     
- Partials      1506     1510       +4
Files with missing lines Coverage Δ
...roid/rum/internal/startup/RumAppStartupDetector.kt 100.00% <ø> (ø)
.../rum/internal/startup/RumAppStartupDetectorImpl.kt 95.45% <100.00%> (-0.29%) ⬇️

... and 42 files with indirect coverage changes

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

@aleksandr-gringauz aleksandr-gringauz marked this pull request as ready for review November 27, 2025 14:32
@aleksandr-gringauz aleksandr-gringauz requested review from a team as code owners November 27, 2025 14:32
@aleksandr-gringauz aleksandr-gringauz merged commit ac0c5d5 into develop Nov 27, 2025
27 of 28 checks passed
@aleksandr-gringauz aleksandr-gringauz deleted the aleksandr-gringauz/RUM-13071/stop-using-process-importance-app-launch branch November 27, 2025 15:05
@aleksandr-gringauz aleksandr-gringauz mentioned this pull request Nov 28, 2025
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@0xnm 0xnm 0xnm 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.

3 participants

@aleksandr-gringauz @codecov-commenter @0xnm