Skip to content

Integrate Datadog Plugin in benchmark application#2618

Merged
ambushwork merged 1 commit into
developfrom
yl/kcp/update-benchmark
Apr 28, 2025
Merged

Integrate Datadog Plugin in benchmark application#2618
ambushwork merged 1 commit into
developfrom
yl/kcp/update-benchmark

Conversation

@ambushwork

Copy link
Copy Markdown
Member

What does this PR do?

Use 1.17.0-rc1 Datadog Plugin in Benchmark application for Compose auto-instrumentation

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 Apr 28, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.06%. Comparing base (ef654e8) to head (2f30bc4).
Report is 140 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2618      +/-   ##
===========================================
- Coverage    70.16%   70.06%   -0.10%     
===========================================
  Files          820      820              
  Lines        30571    30571              
  Branches      5132     5132              
===========================================
- Hits         21449    21418      -31     
- Misses        7708     7724      +16     
- Partials      1414     1429      +15     

see 27 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 April 28, 2025 15:32
@ambushwork
ambushwork requested review from a team as code owners April 28, 2025 15:32
@ambushwork
ambushwork merged commit 5db245f into develop Apr 28, 2025
@ambushwork
ambushwork deleted the yl/kcp/update-benchmark branch April 28, 2025 16:01
BottomNavigationBar(navHostController = navController)
}
) { padding ->
NavigationViewTrackingEffect(

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why was it necessary to remove this?

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.

Because this is the API that we need to auto instrument with kotlin compiler plugin, we need to remove the manual instrumentation here.

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