Skip to content

Add ProfilingFragment into Sample Application#3074

Merged
ambushwork merged 1 commit into
feature/perfetto-profilingfrom
yl/add-profiling-sample
Dec 24, 2025
Merged

Add ProfilingFragment into Sample Application#3074
ambushwork merged 1 commit into
feature/perfetto-profilingfrom
yl/add-profiling-sample

Conversation

@ambushwork

@ambushwork ambushwork commented Dec 23, 2025

Copy link
Copy Markdown
Member

What does this PR do?

Add a sample screen for custom profiling, currently not connected to Profiling feature since custom profiling API is not properly implemented yet, but we can use this screen in synthetics test for now.

image

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/add-profiling-sample branch from dec0c66 to 01d1564 Compare December 23, 2025 13:29
@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Dec 23, 2025

Copy link
Copy Markdown

🎯 Code Coverage
Patch Coverage: 100.00%
Overall Coverage: 66.44% (+0.07%)

View detailed report

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

@ambushwork

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Hooray!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@codecov-commenter

codecov-commenter commented Dec 23, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.27%. Comparing base (4f0483e) to head (86e1421).

Additional details and impacted files
@@                      Coverage Diff                       @@
##           feature/perfetto-profiling    #3074      +/-   ##
==============================================================
- Coverage                       71.28%   71.27%   -0.02%     
==============================================================
  Files                             893      893              
  Lines                           32689    32689              
  Branches                         5493     5493              
==============================================================
- Hits                            23302    23296       -6     
- Misses                           7827     7833       +6     
  Partials                         1560     1560              

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 December 23, 2025 14:16
@ambushwork
ambushwork requested a review from a team as a code owner December 23, 2025 14:16
0xnm
0xnm previously approved these changes Dec 24, 2025
Comment thread sample/kotlin/src/main/res/layout/fragment_profiling.xml Outdated
jonathanmos
jonathanmos previously approved these changes Dec 24, 2025
@ambushwork
ambushwork dismissed stale reviews from jonathanmos and 0xnm via 5c8768c December 24, 2025 09:33
@ambushwork
ambushwork force-pushed the yl/add-profiling-sample branch from 01d1564 to 5c8768c Compare December 24, 2025 09:33
Comment thread sample/kotlin/src/main/res/values/strings.xml Outdated
@ambushwork
ambushwork force-pushed the yl/add-profiling-sample branch from 5c8768c to 86e1421 Compare December 24, 2025 09:50
@ambushwork
ambushwork merged commit 82de848 into feature/perfetto-profiling Dec 24, 2025
26 checks passed
@ambushwork
ambushwork deleted the yl/add-profiling-sample branch December 24, 2025 11:11
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