Skip to content

Remove some unused code#4775

Merged
mstange merged 3 commits into
firefox-devtools:mainfrom
mstange:remove-unused-code
Oct 16, 2023
Merged

Remove some unused code#4775
mstange merged 3 commits into
firefox-devtools:mainfrom
mstange:remove-unused-code

Conversation

@mstange

@mstange mstange commented Oct 13, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@mstange mstange requested a review from canova October 13, 2023 15:15
@mstange mstange self-assigned this Oct 13, 2023
@codecov

codecov Bot commented Oct 13, 2023

Copy link
Copy Markdown

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (e3647d0) 88.30% compared to head (8d4a1f4) 88.29%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4775      +/-   ##
==========================================
- Coverage   88.30%   88.29%   -0.01%     
==========================================
  Files         301      301              
  Lines       26838    26833       -5     
  Branches     7245     7244       -1     
==========================================
- Hits        23698    23693       -5     
  Misses       2924     2924              
  Partials      216      216              
Files Coverage Δ
src/components/shared/thread/SampleGraph.js 97.84% <ø> (-0.11%) ⬇️
src/components/sidebar/CallTreeSidebar.js 86.60% <ø> (ø)
src/components/timeline/TrackThread.js 98.52% <ø> (ø)
src/profile-logic/transforms.js 91.54% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@canova canova left a comment

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.

Thanks for the cleanup!

@mstange mstange merged commit fd79a42 into firefox-devtools:main Oct 16, 2023
@canova canova mentioned this pull request Oct 17, 2023
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