Skip to content

RUM-10173: Change Benchmark artifact Id to dd-sdk-android-benchmark#2686

Merged
ambushwork merged 1 commit into
developfrom
yl/update-benchmark-artifact-name
May 30, 2025
Merged

RUM-10173: Change Benchmark artifact Id to dd-sdk-android-benchmark#2686
ambushwork merged 1 commit into
developfrom
yl/update-benchmark-artifact-name

Conversation

@ambushwork

Copy link
Copy Markdown
Member

What does this PR do?

Change Benchmark artifact Id to dd-sdk-android-benchmark

Motivation

RUM-10173

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 30, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.94%. Comparing base (1e8e26f) to head (ccd7932).
⚠️ Report is 2072 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2686      +/-   ##
===========================================
+ Coverage    69.94%   69.94%   +0.01%     
===========================================
  Files          820      820              
  Lines        30657    30657              
  Branches      5162     5162              
===========================================
+ Hits         21441    21443       +2     
- Misses        7769     7776       +7     
+ Partials      1447     1438       -9     

see 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 marked this pull request as ready for review May 30, 2025 08:58
@ambushwork
ambushwork requested review from a team as code owners May 30, 2025 08:58
0xnm
0xnm previously approved these changes May 30, 2025
Comment thread tools/benchmark/build.gradle.kts Outdated
publishingConfig("An internal benchmarking tool to measure the overhead of Datadog SDK")
publishingConfig(
projectDescription = "An internal benchmarking tool to measure the overhead of Datadog SDK",
customArtifactId = "dd-sdk-android-benchmark"

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.

maybe call it dd-sdk-android-benchmark-internal to highlight that it is for the internal use only?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

updated

@aleksandr-gringauz

Copy link
Copy Markdown
Contributor

@sbarrio jfyi

@ambushwork
ambushwork force-pushed the yl/update-benchmark-artifact-name branch from 1c040fc to ccd7932 Compare May 30, 2025 11:11
@sbarrio

sbarrio commented May 30, 2025

Copy link
Copy Markdown
Contributor

@sbarrio jfyi

Thanks!, I'll modify the import on the RN sdk.

@ambushwork
ambushwork merged commit 34f62bf into develop May 30, 2025
@ambushwork
ambushwork deleted the yl/update-benchmark-artifact-name branch May 30, 2025 12:19
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.

5 participants