Skip to content

Commit

Permalink
feat: add reviewers (#482)
Browse files Browse the repository at this point in the history
Co-authored-by: Arthur Dzieniszewski <arthurdzieniszewski@gmail.com>
  • Loading branch information
gr2m and adzienis committed Dec 22, 2021
1 parent 220e0d8 commit 3a2f600
Show file tree
Hide file tree
Showing 5 changed files with 58 additions and 0 deletions.
1 change: 1 addition & 0 deletions .github/workflows/debug.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,3 +22,4 @@ jobs:
path: "test.txt"
commit-message: "debug"
author: "Gregor Martynus <39992+gr2m@users.noreply.github.com>"
reviewers: gr2m
36 changes: 36 additions & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -209,3 +209,39 @@ jobs:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- if: ${{ fromJson(steps.get-pull-request.outputs.data).title != 'Updated test pull request' }}
run: 'echo "Pull request title is \"${{ fromJson(steps.get-pull-request.outputs.data).title }}\" but expected \"Updated test pull request\"" && exit 1'

addReviewers:
name: "[TEST] Add Reviewers"
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
with:
persist-credentials: false
- run: "date > test.txt"
- run: "npm ci"
- run: "npm run build"
- uses: ./
id: run
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
ACTIONS_STEP_DEBUG: true
with:
title: Test Add Reviewers
body: This pull request is part of the CI - please ignore.
branch: test-add-reviewers-${{ github.run_number }}
commit-message: "Just testing [skip ci]"
reviewers: gr2m
- uses: octokit/request-action@v2.x
id: get-pull-request
with:
route: GET /repos/{owner}/{repo}/pulls/{pull_number}
owner: gr2m
repo: create-or-update-pull-request-action
pull_number: ${{ steps.run.outputs.pull-request-number }}
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- run: "git push https://x-access-token:${GITHUB_TOKEN}@github.com/${GITHUB_REPOSITORY}.git :test-add-reviewers-${{ github.run_number }}"
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- if: ${{ !contains(toJson(fromJson(steps.get-pull-request.outputs.data).requested_reviewers.*.login), 'gr2m') }}
run: 'echo "Requested reviewers are \"${{ toJson(fromJson(steps.get-pull-request.outputs.data).requested_reviewers.*.login) }}\" but expected \"[\n gr2m \n]\"" && exit 1'
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ with:
author: "Lorem J. Ipsum <lorem@example.com>"
labels: label1, label2
assignees: user1, user2
reviewers: user1, user2
auto-merge: squash
update-pull-request-title-and-body: false
```
Expand Down
3 changes: 3 additions & 0 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,9 @@ inputs:
assignees:
description: Comma separated list of assignees to apply to the pull request
required: false
reviewers:
description: Comma separated list of reviewers to apply to the pull request
required: false
auto-merge:
description: "Enable auto merge for pull request. Requires auto merging to be enabled in repository settings"
required: false
Expand Down
17 changes: 17 additions & 0 deletions index.js
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ async function main() {
author: core.getInput("author"),
labels: core.getInput("labels"),
assignees: core.getInput("assignees"),
reviewers: core.getInput("reviewers"),
autoMerge: core.getInput("auto-merge"),
updatePRTitleAndBody: core.getInput("update-pull-request-title-and-body"),
};
Expand Down Expand Up @@ -221,6 +222,22 @@ async function main() {
core.info(`Assignees added: ${assignees.join(", ")}`);
core.debug(inspect(data));
}

if (inputs.reviewers) {
core.debug(`Adding reviewers: ${inputs.reviewers}`);
const reviewers = inputs.reviewers.trim().split(/\s*,\s*/);
const { data } = await octokit.request(
`POST /repos/{owner}/{repo}/pulls/{pull_number}/requested_reviewers`,
{
owner,
repo,
pull_number: number,
reviewers,
}
);
core.info(`Reviewers added: ${reviewers.join(", ")}`);
core.debug(inspect(data));
}

if (inputs.autoMerge) {
const query = `
Expand Down

0 comments on commit 3a2f600

Please sign in to comment.