search

skymoore / required-approvals

Github Action to check PR approvals and codeowners
7 stars 6 forks source link

Include reason in output #15

Open cjlludwig opened 2 months ago

cjlludwig commented 2 months ago

Love the action! One improvement I think would be a nice to have is exposing the reason for not meeting approval requirements. Those are logged today about could easily be added to a separate output, outputs.reason or something similar.

This would give consumers more flexibility around handling failures, like commenting back on PRs with reason.

skymoore commented 1 month ago

@cjlludwig Thank you for the suggestion. I will get to this soon if someone else doesn't :).