Skip to content

Commit 6f69cc2

Browse files
authored
chore: refine reviewer auto-assignment triggers and routing (#8709)
* chore: remove pubsub and bigtable from path routing for reviewer auto-assignment * chore: remove cloud-sdk-nodejs-team from reviewers if specific reviewer is assigned * chore: trigger auto-assign reviewers workflow on reopened pull requests
1 parent c9263b6 commit 6f69cc2

1 file changed

Lines changed: 16 additions & 3 deletions

File tree

.github/workflows/assign-reviewers.yml

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name: Auto Assign Reviewers
22

33
on:
44
pull_request:
5-
types: [opened, ready_for_review]
5+
types: [opened, ready_for_review, reopened]
66

77
permissions:
88
pull-requests: write
@@ -79,8 +79,6 @@ jobs:
7979
{ prefix: 'handwritten/firestore/', team: 'firestore-team' },
8080
{ prefix: 'handwritten/spanner/', team: 'spanner-team' },
8181
{ prefix: 'handwritten/bigquery-storage/', team: 'bigquery-team' },
82-
{ prefix: 'handwritten/pubsub/', team: 'pubsub-team' },
83-
{ prefix: 'handwritten/bigtable/', team: 'bigtable-team' },
8482
{ prefix: 'core/packages/google-auth-library-nodejs/', team: 'aion-team' }
8583
];
8684
@@ -199,10 +197,25 @@ jobs:
199197
pull_number: context.payload.pull_request.number,
200198
reviewers: [leastLoadedReviewer],
201199
});
200+
assigned = true;
202201
} catch (err) {
203202
console.error("Failed to assign reviewers using load balancing:", err);
204203
}
205204
} else {
206205
console.warn("No default reviewers found in CODEOWNERS (or they only contained teams/author).");
207206
}
208207
}
208+
209+
if (assigned) {
210+
console.log("Removing default cloud-sdk-nodejs-team reviewer...");
211+
try {
212+
await github.rest.pulls.removeRequestedReviewers({
213+
owner: context.repo.owner,
214+
repo: context.repo.repo,
215+
pull_number: context.payload.pull_request.number,
216+
team_reviewers: ['cloud-sdk-nodejs-team'],
217+
});
218+
} catch (err) {
219+
console.warn("Failed to remove default cloud-sdk-nodejs-team reviewer:", err.message || err);
220+
}
221+
}

0 commit comments

Comments
 (0)