feat: add "delete on status" option#126
Conversation
allows deleting the comment if a certain status has been reached. Useful for removing comments added on failure when success status is reached
|
@mshick Could we get this reviewed and merged? |
|
hey @mshick, big fan of this action! Do you think we could get a review on this PR? It'd be super helpful to have this functionality. |
|
Hi all, thanks for the contribution @manan-jadhav-ab . Sorry I'm so long in getting to this — over a year! I'll review right away and maybe one of you would be interested in joining as a maintainer? There's not much to do, but I find I don't have a ton of time to commit lately. If you'd be interested just send me an email (address in my profile). Thanks! |
|
@manan-jadhav-ab Everything looks good here. If you can resolve the conflict I'll get this merged up. |
✅ |
mshick
left a comment
There was a problem hiding this comment.
A couple small items, and a new merge conflict to address.
Co-authored-by: Michael Shick <m@shick.us>
|
@manan-jadhav-ab I've never figured out how to get some of these actions to run for a contrib PR. Going to open my own copy of this to make sure CI passes, then can merge it. |
|
@all-contributors add @manan-jadhav-ab for code |
|
I've put up a pull request to add @manan-jadhav-ab! 🎉 |
|
closes #108 |
|
Closes #123 |
allows deleting the comment if a certain status has been reached. Useful for removing comments added on failure when success status is reached