Skip to content

Conversation

ambushwork
Copy link
Member

What does this PR do?

Create profiling module

Motivation

RUM-11830

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)

@ambushwork ambushwork force-pushed the yl/profiling/create-module branch 3 times, most recently from 586a44c to e2c2547 Compare September 23, 2025 16:02
@codecov-commenter
Copy link

codecov-commenter commented Sep 23, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 70.89%. Comparing base (d20aaa2) to head (08b5702).

Files with missing lines Patch % Lines
...in/java/com/datadog/android/profiling/Profiling.kt 0.00% 1 Missing ⚠️
Additional details and impacted files
@@                      Coverage Diff                       @@
##           feature/perfetto-profiling    #2893      +/-   ##
==============================================================
- Coverage                       71.11%   70.89%   -0.22%     
==============================================================
  Files                             810      811       +1     
  Lines                           29677    29678       +1     
  Branches                         4967     4967              
==============================================================
- Hits                            21104    21038      -66     
- Misses                           7230     7285      +55     
- Partials                         1343     1355      +12     
Files with missing lines Coverage Δ
...in/java/com/datadog/android/profiling/Profiling.kt 0.00% <0.00%> (ø)

... and 37 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 force-pushed the yl/profiling/create-module branch from e2c2547 to 1495bbf Compare September 23, 2025 16:53
@ambushwork ambushwork marked this pull request as ready for review September 23, 2025 16:53
@ambushwork ambushwork requested review from a team as code owners September 23, 2025 16:53
0xnm
0xnm previously approved these changes Sep 24, 2025
Comment on lines 44 to 45
implementation(libs.gson)
implementation(libs.okHttp)
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure these 2 will be needed. It is better to remove them and add only in needed.

Same maybe for dd-sdk-android-internal usage above so far.

Comment on lines 61 to 62
testImplementation(libs.okHttp)
testImplementation(libs.okHttpMock)
Copy link
Member

Choose a reason for hiding this comment

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

probably can be removed as well

@ambushwork ambushwork dismissed stale reviews from aleksandr-gringauz and 0xnm via 08b5702 September 24, 2025 08:25
@ambushwork ambushwork force-pushed the yl/profiling/create-module branch from 1495bbf to 08b5702 Compare September 24, 2025 08:25
0xnm
0xnm previously approved these changes Sep 24, 2025
api(project(":dd-sdk-android-core"))
implementation(project(":dd-sdk-android-internal"))
implementation(libs.kotlin)
implementation(libs.gson)
Copy link
Member

Choose a reason for hiding this comment

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

will this one be needed?

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