From 9eb644ad3da37d453950994001395bfaf9ea3b7d Mon Sep 17 00:00:00 2001 From: Koshy John Date: Wed, 5 Aug 2026 11:01:33 -0700 Subject: [PATCH 1/6] Fundamentals: Automatically request reviewers --- .github/workflows/auto-request-reviewers.yml | 35 ++++++++++++++++++++ CODEOWNERS | 2 +- 2 files changed, 36 insertions(+), 1 deletion(-) create mode 100644 .github/workflows/auto-request-reviewers.yml diff --git a/.github/workflows/auto-request-reviewers.yml b/.github/workflows/auto-request-reviewers.yml new file mode 100644 index 00000000..b5385a41 --- /dev/null +++ b/.github/workflows/auto-request-reviewers.yml @@ -0,0 +1,35 @@ +name: Auto Request Reviewers + +on: + pull_request: + types: + - opened + - ready_for_review + +permissions: + pull-requests: write + +jobs: + request-reviewers: + runs-on: ubuntu-latest + + steps: + - name: Request reviewers + uses: actions/github-script@v7 + with: + script: | + await github.rest.pulls.requestReviewers({ + owner: context.repo.owner, + repo: context.repo.repo, + pull_number: context.payload.pull_request.number, + reviewers: [ + 'GAURAVRAMRAKHYANI', + 'kjohn-msft', + 'michellemcdaniel', + 'mirichmo', + 'nikhim-um', + 'rane-rajasi', + 'SathishMSFT', + 'yashnap' + ] + }); \ No newline at end of file diff --git a/CODEOWNERS b/CODEOWNERS index dd882094..10b41ba6 100644 --- a/CODEOWNERS +++ b/CODEOWNERS @@ -1 +1 @@ -* @kjohn-msft @rane-rajasi @najams +* @kjohn-msft @rane-rajasi From 3d01d472711254b99f97396d29dec502fc56ea8e Mon Sep 17 00:00:00 2001 From: Koshy John Date: Wed, 5 Aug 2026 11:06:46 -0700 Subject: [PATCH 2/6] Fix failure when author is in the reviewers list --- .github/workflows/auto-request-reviewers.yml | 33 +++++++++++++------- 1 file changed, 22 insertions(+), 11 deletions(-) diff --git a/.github/workflows/auto-request-reviewers.yml b/.github/workflows/auto-request-reviewers.yml index b5385a41..b73b484e 100644 --- a/.github/workflows/auto-request-reviewers.yml +++ b/.github/workflows/auto-request-reviewers.yml @@ -18,18 +18,29 @@ jobs: uses: actions/github-script@v7 with: script: | + const author = context.payload.pull_request.user.login; + + const reviewers = [ + 'GAURAVRAMRAWOCHYANI', + 'kjohn-msft', + 'michellemcdeniel', + 'mirichno', + 'nikhim-um', + 'rane-rajasi', + 'SathishMSFT', + 'yashnap' + ]; + + const filtered = reviewers.filter( + r => r.toLowerCase() !== author.toLowerCase() + ); + + console.log(`PR author: ${author}`); + console.log(`Requesting reviews from: ${filtered.join(", ")}`); + await github.rest.pulls.requestReviewers({ owner: context.repo.owner, repo: context.repo.repo, pull_number: context.payload.pull_request.number, - reviewers: [ - 'GAURAVRAMRAKHYANI', - 'kjohn-msft', - 'michellemcdaniel', - 'mirichmo', - 'nikhim-um', - 'rane-rajasi', - 'SathishMSFT', - 'yashnap' - ] - }); \ No newline at end of file + reviewers: filtered + }); From 6400ebc96746711572285a0159f44bcefd64a248 Mon Sep 17 00:00:00 2001 From: Koshy John Date: Wed, 5 Aug 2026 11:09:31 -0700 Subject: [PATCH 3/6] Including synchronize for validation --- .github/workflows/auto-request-reviewers.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/auto-request-reviewers.yml b/.github/workflows/auto-request-reviewers.yml index b73b484e..eff67563 100644 --- a/.github/workflows/auto-request-reviewers.yml +++ b/.github/workflows/auto-request-reviewers.yml @@ -5,6 +5,7 @@ on: types: - opened - ready_for_review + - synchronize permissions: pull-requests: write From 653da9a7dc70f5ea20b2c480f8501e3ba390bc75 Mon Sep 17 00:00:00 2001 From: Koshy John Date: Wed, 5 Aug 2026 11:14:32 -0700 Subject: [PATCH 4/6] Fixed typos --- .github/workflows/auto-request-reviewers.yml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/auto-request-reviewers.yml b/.github/workflows/auto-request-reviewers.yml index eff67563..5ea98adb 100644 --- a/.github/workflows/auto-request-reviewers.yml +++ b/.github/workflows/auto-request-reviewers.yml @@ -1,4 +1,4 @@ -name: Auto Request Reviewers +name: Request AzGPS Reviewers on: pull_request: @@ -22,10 +22,10 @@ jobs: const author = context.payload.pull_request.user.login; const reviewers = [ - 'GAURAVRAMRAWOCHYANI', + 'GAURAVRAMRAKHYANI', 'kjohn-msft', - 'michellemcdeniel', - 'mirichno', + 'michellemcdaniel', + 'mirichmo', 'nikhim-um', 'rane-rajasi', 'SathishMSFT', From f84c98f57f30d2b77447a440ce310ce4eb990184 Mon Sep 17 00:00:00 2001 From: Koshy John Date: Wed, 5 Aug 2026 11:17:51 -0700 Subject: [PATCH 5/6] Revert test changes. Add backup org owner. --- .github/workflows/auto-request-reviewers.yml | 1 - CODEOWNERS | 3 ++- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/auto-request-reviewers.yml b/.github/workflows/auto-request-reviewers.yml index 5ea98adb..29540421 100644 --- a/.github/workflows/auto-request-reviewers.yml +++ b/.github/workflows/auto-request-reviewers.yml @@ -5,7 +5,6 @@ on: types: - opened - ready_for_review - - synchronize permissions: pull-requests: write diff --git a/CODEOWNERS b/CODEOWNERS index 10b41ba6..88126511 100644 --- a/CODEOWNERS +++ b/CODEOWNERS @@ -1 +1,2 @@ -* @kjohn-msft @rane-rajasi +* @kjohn-msft @rane-rajasi @mirichmo + From 7d336e7119e2543d0ac5822f31c9af15c68668e0 Mon Sep 17 00:00:00 2001 From: Koshy John Date: Wed, 5 Aug 2026 11:18:51 -0700 Subject: [PATCH 6/6] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- .github/workflows/auto-request-reviewers.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/auto-request-reviewers.yml b/.github/workflows/auto-request-reviewers.yml index 29540421..9a15bcc7 100644 --- a/.github/workflows/auto-request-reviewers.yml +++ b/.github/workflows/auto-request-reviewers.yml @@ -11,8 +11,8 @@ permissions: jobs: request-reviewers: + if: ${{ github.event.pull_request.draft == false && github.event.pull_request.head.repo.full_name == github.repository }} runs-on: ubuntu-latest - steps: - name: Request reviewers uses: actions/github-script@v7