Skip to content

Commit 1ea959a

Browse files
authored
ci: Don't add status label to prs automatically (#58606)
This status label will be helpful once it has a well defined semantic and we have more statuses to transition between. As is, it is mostly noise and I think it should be (temporarily) removed.
1 parent 050284c commit 1ea959a

File tree

1 file changed

+10
-9
lines changed

1 file changed

+10
-9
lines changed

.github/workflows/PrAssignee.yml

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -172,15 +172,16 @@ jobs:
172172
assignees: selectedAssignee,
173173
});
174174
175-
// Add the "pr review" label
176-
const prReviewLabel = 'status: waiting for PR reviewer';
177-
console.log('Attempting to add prReviewLabel to this PR...');
178-
await github.rest.issues.addLabels({
179-
owner: context.repo.owner,
180-
repo: context.repo.repo,
181-
issue_number: context.payload.pull_request.number,
182-
labels: [prReviewLabel],
183-
});
175+
// The following is commented out because the label only makes sense in the presence of a larger state machine
176+
// // Add the "pr review" label
177+
// const prReviewLabel = 'status: waiting for PR reviewer';
178+
// console.log('Attempting to add prReviewLabel to this PR...');
179+
// await github.rest.issues.addLabels({
180+
// owner: context.repo.owner,
181+
// repo: context.repo.repo,
182+
// issue_number: context.payload.pull_request.number,
183+
// labels: [prReviewLabel],
184+
// });
184185
185186
// Now get the updated PR info, and see if we were successful:
186187
const updatedPrData = await github.rest.pulls.get({

0 commit comments

Comments
 (0)