From 46fe963c4e1aeafaebe448bfcf78663c917f1db3 Mon Sep 17 00:00:00 2001 From: SlyCaptainFlint Date: Wed, 30 Nov 2022 20:52:46 -0800 Subject: [PATCH] Move close-pr workflow from react-native-bot to actions/github-script@v6 (#35500) Summary: This change adds a new action and workflow that will run whenever a commit is pushed that closes a PR via its message. When this happens, the corresponding PR will be labeled with "Merged" and a comment will be added with the name of the committer and the sha of the commit that resolved the PR. This is not new functionality - it merely replaces a portion of the react-native-bot's functionality with a GitHub action. ## Changelog [Internal] [Changed] - Moved close PR actions to a GitHub action Pull Request resolved: https://github.com/facebook/react-native/pull/35500 Test Plan: Tested the golden path here ("Merged" label and comment added): https://github.com/SlyCaptainFlint/react-native/pull/27 Tested the case where a PR already labeled with Merged was processed and no new comment was left on it (simulates the case where react-native-bot, which is still going to be running, got to the PR and processed it before the new GitHub action did): https://github.com/SlyCaptainFlint/react-native/pull/26 Reviewed By: cipolleschi Differential Revision: D41567047 Pulled By: SlyCaptainFlint fbshipit-source-id: 23745a17950b75c1a8f25f0b840c2172332c4cfa --- .github/workflows/close_pr.yml | 62 ++++++++++++++++++++++++++++++++++ 1 file changed, 62 insertions(+) create mode 100644 .github/workflows/close_pr.yml diff --git a/.github/workflows/close_pr.yml b/.github/workflows/close_pr.yml new file mode 100644 index 00000000000..a31aa01ce73 --- /dev/null +++ b/.github/workflows/close_pr.yml @@ -0,0 +1,62 @@ +name: Label closed PR as merged and leave a comment +on: + push + +permissions: + contents: read + pull-requests: write + +jobs: + comment-and-label: + runs-on: ubuntu-latest + steps: + - uses: actions/github-script@v6 + with: + github-token: ${{ secrets.GITHUB_TOKEN }} + script: | + if(!context.payload.commits || !context.payload.commits.length) return; + const sha = context.payload.commits[0].id; + + const {commit, author} = (await github.rest.repos.getCommit({ + ref: sha, + owner: context.repo.owner, + repo: context.repo.repo, + })).data; + + // Looking at the commit message, checks which PR number, if any, was closed by this commit + const getClosedPrIfExists = (commit) => { + if(!commit || !commit.message) return; + const prClosingRegex = /Closes https:\/\/github.com\/facebook\/react-native\/pull\/([0-9]+)|Pull Request resolved: https:\/\/github.com\/facebook\/react-native\/pull\/([0-9]+)/; + const prClosingMatch = commit.message.match(prClosingRegex); + if(!prClosingMatch || (!prClosingMatch[1] && ! prClosingMatch[2])) return; + return prClosingMatch[1] ?? prClosingMatch[2]; + }; + + const closedPrNumber = getClosedPrIfExists(commit); + if(!closedPrNumber) return; + + const pr = (await github.rest.pulls.get({ + pull_number: closedPrNumber, + owner: context.repo.owner, + repo: context.repo.repo, + })).data; + + // If the PR has already been processed (labeled as Merged), skip it + const mergedLabel = "Merged"; + if(pr.labels && pr.labels.some(label => label.name === mergedLabel)) return; + + const authorName = author?.login ? `@${author.login}` : commit.author.name; + + github.rest.issues.createComment({ + issue_number: closedPrNumber, + owner: context.repo.owner, + repo: context.repo.repo, + body: `This pull request was successfully merged by ${authorName} in **${sha}**.\n\n[When will my fix make it into a release?](https://github.com/facebook/react-native/wiki/Release-FAQ#when-will-my-fix-make-it-into-a-release) | [Upcoming Releases](https://github.com/reactwg/react-native-releases/discussions/categories/releases)` + }); + + github.rest.issues.addLabels({ + issue_number: closedPrNumber, + owner: context.repo.owner, + repo: context.repo.repo, + labels: [mergedLabel] + });