Skip to content

Commit cded13f

Browse files
committed
fix: apply linux status labels directly instead of via deferred labelsToAdd set
1 parent c25e4da commit cded13f

1 file changed

Lines changed: 36 additions & 4 deletions

File tree

.github/workflows/labeler.yml

Lines changed: 36 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1024,9 +1024,29 @@ jobs:
10241024
}
10251025
}
10261026
}
1027-
labelsToAdd.add(NO_LINUX_LABEL);
1027+
if (!existingLabels.has(NO_LINUX_LABEL)) {
1028+
try {
1029+
await github.rest.issues.addLabels({ owner, repo, issue_number: issueNumber, labels: [NO_LINUX_LABEL] });
1030+
console.log(`Added label: ${NO_LINUX_LABEL}`);
1031+
} catch (err) {
1032+
console.log(`Could not add label "${NO_LINUX_LABEL}": ${err.message}`);
1033+
}
1034+
}
1035+
if (existingLabels.has(CONFIRMED_LINUX_LABEL)) {
1036+
try {
1037+
await github.rest.issues.removeLabel({ owner, repo, issue_number: issueNumber, name: CONFIRMED_LINUX_LABEL });
1038+
console.log(`Removed label: ${CONFIRMED_LINUX_LABEL}`);
1039+
} catch (err) {
1040+
console.log(`Could not remove label "${CONFIRMED_LINUX_LABEL}": ${err.message}`);
1041+
}
1042+
}
10281043
} else if (existingLabels.has(NO_LINUX_LABEL)) {
1029-
labelsToRemove.add(NO_LINUX_LABEL);
1044+
try {
1045+
await github.rest.issues.removeLabel({ owner, repo, issue_number: issueNumber, name: NO_LINUX_LABEL });
1046+
console.log(`Removed label: ${NO_LINUX_LABEL}`);
1047+
} catch (err) {
1048+
console.log(`Could not remove label "${NO_LINUX_LABEL}": ${err.message}`);
1049+
}
10301050
}
10311051
10321052
if (shouldApplyConfirmedLinuxLabel) {
@@ -1047,9 +1067,21 @@ jobs:
10471067
}
10481068
}
10491069
}
1050-
labelsToAdd.add(CONFIRMED_LINUX_LABEL);
1070+
if (!existingLabels.has(CONFIRMED_LINUX_LABEL)) {
1071+
try {
1072+
await github.rest.issues.addLabels({ owner, repo, issue_number: issueNumber, labels: [CONFIRMED_LINUX_LABEL] });
1073+
console.log(`Added label: ${CONFIRMED_LINUX_LABEL}`);
1074+
} catch (err) {
1075+
console.log(`Could not add label "${CONFIRMED_LINUX_LABEL}": ${err.message}`);
1076+
}
1077+
}
10511078
} else if (existingLabels.has(CONFIRMED_LINUX_LABEL)) {
1052-
labelsToRemove.add(CONFIRMED_LINUX_LABEL);
1079+
try {
1080+
await github.rest.issues.removeLabel({ owner, repo, issue_number: issueNumber, name: CONFIRMED_LINUX_LABEL });
1081+
console.log(`Removed label: ${CONFIRMED_LINUX_LABEL}`);
1082+
} catch (err) {
1083+
console.log(`Could not remove label "${CONFIRMED_LINUX_LABEL}": ${err.message}`);
1084+
}
10531085
}
10541086
10551087
const reasons = [];

0 commit comments

Comments
 (0)