Skip to content

Conversation

@QuiiBz
Copy link
Contributor

@QuiiBz QuiiBz commented May 9, 2022

This library has been forked from use-media, many thanks to the original author, Vadim Dalecky.

@codecov
Copy link

codecov bot commented May 9, 2022

Codecov Report

Merging #766 (722f6dc) into main (498d337) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #766   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           20        21    +1     
  Lines          531       551   +20     
  Branches        74        77    +3     
=========================================
+ Hits           531       551   +20     
Impacted Files Coverage Δ
packages/use-media/src/useMedia.ts 100.00% <100.00%> (ø)

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 498d337...722f6dc. Read the comment docs.

@QuiiBz QuiiBz force-pushed the feat/add-use-media branch from 2f01d91 to 01a5b36 Compare May 10, 2022 08:08
@QuiiBz QuiiBz marked this pull request as ready for review May 10, 2022 09:05
@QuiiBz QuiiBz requested a review from chambo-e May 11, 2022 14:50
Copy link
Member

@DorianMaliszewski DorianMaliszewski left a comment

Choose a reason for hiding this comment

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

Need 100% coverage :)

@QuiiBz QuiiBz force-pushed the feat/add-use-media branch from 86a80e4 to f28ea15 Compare June 6, 2022 12:45
@QuiiBz QuiiBz requested a review from DorianMaliszewski June 6, 2022 13:38
@QuiiBz QuiiBz force-pushed the feat/add-use-media branch from f28ea15 to 722f6dc Compare June 7, 2022 08:02
@QuiiBz QuiiBz enabled auto-merge (squash) June 7, 2022 08:02
@QuiiBz QuiiBz merged commit 34c56f6 into main Jun 7, 2022
@QuiiBz QuiiBz deleted the feat/add-use-media branch June 7, 2022 08: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.

4 participants