Skip to content

Remove some incorrect conditions from local track isSelected prop#4143

Merged
julienw merged 1 commit into
firefox-devtools:mainfrom
canova:remove-incorrect-checks
Jul 18, 2022
Merged

Remove some incorrect conditions from local track isSelected prop#4143
julienw merged 1 commit into
firefox-devtools:mainfrom
canova:remove-incorrect-checks

Conversation

@canova

@canova canova commented Jul 14, 2022

Copy link
Copy Markdown
Member

We don't have an event-delay tab, therefore these conditions will always return true.

We don't have an event-delay tab, therefore these conditions will
always return true.
@canova canova requested a review from julienw July 14, 2022 12:05
@codecov

codecov Bot commented Jul 14, 2022

Copy link
Copy Markdown

Codecov Report

Merging #4143 (bd3cea6) into main (0b9f161) will increase coverage by 0.01%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main    #4143      +/-   ##
==========================================
+ Coverage   88.26%   88.27%   +0.01%     
==========================================
  Files         280      280              
  Lines       24392    24388       -4     
  Branches     6480     6477       -3     
==========================================
- Hits        21530    21529       -1     
+ Misses       2658     2655       -3     
  Partials      204      204              
Impacted Files Coverage Δ
src/components/timeline/LocalTrack.js 71.95% <50.00%> (+2.18%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0b9f161...bd3cea6. Read the comment docs.

@julienw julienw left a comment

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.

thanks for the clean-up!

@julienw julienw merged commit a9220a8 into firefox-devtools:main Jul 18, 2022
@canova canova deleted the remove-incorrect-checks branch July 18, 2022 12:23
@canova canova mentioned this pull request Jul 26, 2022
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