search

anuraag016 / Jest-Coverage-Diff

Jest Coverage report diff as a comment for Pull requests
MIT License
59 stars 46 forks source link

add ability to compare total coverage number against new total_delta fixes resolves #37 #36

Closed datajoy-eric closed 2 years ago

datajoy-eric commented 2 years ago

Adds the ability to be able to compare the total coverage changes in a change against a new total_delta value

The delta value was per file; however there are times where someone may be interested in only comparing against the total changes, and not so much at a per file granularity. Or if they wish to have different deltas (i.e. a smaller delta repo wise and larger delta at a file level)

would resolve: https://github.com/anuraag016/Jest-Coverage-Diff/issues/37

datajoy-eric commented 2 years ago

Hello @anuraag016 @sanjeev9160 ,

Was wondering if you could give this PR a review :)

Thanks so much!

datajoy-eric commented 2 years ago

Hello @anuraag016 @sanjeev9160

Just checking in if possible you could give this PR a review :) thanks!

anuraag016 commented 2 years ago

Hello @anuraag016 @sanjeev9160

Just checking in if possible you could give this PR a review :) thanks!

Had missed this earlier, thanks for being patient and following up on this 😄

datajoy-eric commented 2 years ago

Hey @anuraag016, no worries at all! Seems like I missed this being merged for a bit now too, but thanks for the review and merging it in! 😄