Skip to content

Commit dbc675b

Browse files
committed
cleanup workflow
1 parent 3049905 commit dbc675b

File tree

1 file changed

+21
-9
lines changed

1 file changed

+21
-9
lines changed

.github/workflows/random-reviewer-assignment.yml

Lines changed: 21 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ permissions:
88
pull-requests: write
99

1010
env:
11-
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
11+
GITHUB_TOKEN: ${{ secrets.GH_TOKEN_FOR_PR_ASSIGNMENT }}
1212

1313
jobs:
1414
assign-reviewer:
@@ -18,34 +18,46 @@ jobs:
1818
uses: actions/github-script@v6
1919
with:
2020
script: |
21+
const TRIAGE_USERNAME = 'Lagoja';
22+
2123
try {
22-
// Get team members
2324
const teamMembers = await github.rest.teams.listMembersInOrg({
2425
org: 'jetify-com',
2526
team_slug: 'eng'
2627
});
2728
28-
// Exclude PR author from potential reviewers
29-
const prAuthor = context.payload.pull_request.user.login;
29+
const prAuthor = context.payload.pull_request.user.login.toLowerCase();
30+
console.log(`PR Author: ${prAuthor}`);
31+
32+
// Get eligible reviewers (excluding PR author and lagoja)
33+
3034
const eligibleReviewers = teamMembers.data
3135
.map(member => member.login)
32-
.filter(login => login !== prAuthor)
33-
.filter(login => login !== "lagoja");
36+
.filter(login => {
37+
const loginLower = login.toLowerCase();
38+
return loginLower !== prAuthor && loginLower !== TRIAGE_USERNAME;
39+
});
3440
3541
if (eligibleReviewers.length === 0) {
3642
console.log('No eligible reviewers found');
3743
return;
3844
}
3945
40-
// Randomly select a reviewer
46+
// Build reviewers list: random reviewer + lagoja (if not PR author)
4147
const randomReviewer = eligibleReviewers[Math.floor(Math.random() * eligibleReviewers.length)];
42-
console.log(`Assigning reviewer: ${randomReviewer}`);
48+
const reviewers = [randomReviewer];
49+
50+
if (prAuthor !== TRIAGE_USERNAME) {
51+
reviewers.push(TRIAGE_USERNAME);
52+
}
53+
54+
console.log(`Assigning reviewers: ${reviewers.join(', ')}`);
4355
4456
await github.rest.pulls.requestReviewers({
4557
owner: context.repo.owner,
4658
repo: context.repo.repo,
4759
pull_number: context.payload.pull_request.number,
48-
reviewers: [randomReviewer, "lagoja"]
60+
reviewers
4961
});
5062
5163
} catch (error) {

0 commit comments

Comments
 (0)