Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to manually end UI load trace #1776

Merged
merged 1 commit into from
Dec 20, 2024
Merged

Add option to manually end UI load trace #1776

merged 1 commit into from
Dec 20, 2024

Conversation

bidetofevil
Copy link
Collaborator

@bidetofevil bidetofevil commented Dec 20, 2024

Goal

Add in the ability at the UiLoadTraceEmitter level to designate a UI Load trace to be ended manually. This has not been hooked up with the API yet so it cannot be triggered externally yet.

Copy link
Collaborator Author

bidetofevil commented Dec 20, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link

codecov bot commented Dec 20, 2024

Codecov Report

Attention: Patch coverage is 83.33333% with 6 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@eac60be). Learn more about missing BASE report.
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...dk/internal/capture/activity/UiLoadTraceEmitter.kt 81.25% 4 Missing and 2 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1776   +/-   ##
=======================================
  Coverage        ?   85.44%           
=======================================
  Files           ?      466           
  Lines           ?    10979           
  Branches        ?     1631           
=======================================
  Hits            ?     9381           
  Misses          ?      867           
  Partials        ?      731           
Files with missing lines Coverage Δ
.../embracesdk/internal/capture/activity/UiLoadExt.kt 100.00% <100.00%> (ø)
...dk/internal/capture/activity/UiLoadTraceEmitter.kt 88.43% <81.25%> (ø)

@bidetofevil bidetofevil marked this pull request as ready for review December 20, 2024 19:50
@bidetofevil bidetofevil requested a review from a team as a code owner December 20, 2024 19:50
@bidetofevil bidetofevil requested review from fractalwrench and removed request for a team December 20, 2024 19:50
Copy link
Contributor

@fractalwrench fractalwrench left a comment

Choose a reason for hiding this comment

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

LGTM

@bidetofevil bidetofevil changed the base branch from hho/ui-load-interrupt to graphite-base/1776 December 20, 2024 21:51
@bidetofevil bidetofevil changed the base branch from graphite-base/1776 to main December 20, 2024 21:52
Copy link
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

@bidetofevil bidetofevil merged commit 9e471a8 into main Dec 20, 2024
7 of 8 checks passed
Copy link
Collaborator Author

Merge activity

  • Dec 20, 5:03 PM EST: A user merged this pull request with Graphite.

@bidetofevil bidetofevil deleted the hho/manual-end branch December 20, 2024 22:03
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.

2 participants