action-get-merged-pull-request icon indicating copy to clipboard operation
action-get-merged-pull-request copied to clipboard

Add PR author to outputs

Open roryabraham opened this issue 3 years ago • 4 comments

What this PR does / Why we need it

Adds the PR author to the outputs of this action. I am building a CI/CD flow where I want to automerge certain pull requests (only those created by one of our bot users). I'm already using this action, and just need another output to check the actor who created the pull request. 😁

Which issue(s) this PR fixes

n/a

roryabraham avatar Jun 03 '21 18:06 roryabraham

Just looking at the commit history, it looks like there have been many dependabot dependency updates since the last JS change in this repo. I think that's why there's such a large diff in the compiled ncc output.

roryabraham avatar Jun 03 '21 18:06 roryabraham

@micnncim Any interest in merging this pull request? FWIW I've been using it in production for about 3 months now: https://github.com/Expensify/App/blob/main/.github/workflows/deploy.yml#L17

roryabraham avatar Sep 02 '21 05:09 roryabraham

I've updated this pull request to include outputs for merged_by and html_url.

roryabraham avatar Dec 22 '21 18:12 roryabraham

@micnncim Hello! Would you be able to consider merging this PR into the main codebase? Thanks :)

mountiny avatar Mar 15 '22 15:03 mountiny