search

anuraag016 / Jest-Coverage-Diff

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

feat: Use Same Comment for Updates #27

Closed therynamo closed 2 years ago

therynamo commented 3 years ago

Hey there! Thanks for building out this coverage diff action!

I was using jest-coverage-reporting-action previously and it was taking an exceptionally long time to run. I gave your action a shot and it was far faster, so 👏 . One thing that was nice though - was their ability to comment back to the same issue. I think that would be a valuable add here so that PRs aren't receiving a new comment with every commit.

Let me know what you think - I'd be happy to try and take a stab at it too if you don't have the bandwidth.

anuraag016 commented 3 years ago

Hey @therynamo! Thanks a lot for opening this issue, you have a valid point here, its worth having this feature as an option so that it can be used as required. I remember using one single comment thing for one of my test action, I might still have that code somewhere. Let me see if I can salvage the same code and use it here.

therynamo commented 3 years ago

Awesome @anuraag016 - excited to see this - let me know if I can help in any way. 👍

anuraag016 commented 3 years ago

Raised a pr #30 for adding this feature, by default it would be turned off, as most users would be used to having different comments for different commits. Using action input it can be turned on. Once turned on, the action will always try to update the old comments rather than creating new