Просмотр исходного кода

fix(triage): match Linux check verdict to a maintainer-set label (#5065)

When a maintainer has set the Linux support label manually, the check
comment kept printing the automatic verdict, which could contradict the
label (e.g. "does not appear to have native Linux support" alongside
"linux support confirmed"). Derive the verdict from the kept label in
that case; the automatic evidence list is unchanged.
Daniel Gibbs 2 дней назад
Родитель
Сommit
daad25241b
1 измененных файлов с 5 добавлено и 1 удалено
  1. 5 1
      .github/workflows/action-issue-triage-automation.yml

+ 5 - 1
.github/workflows/action-issue-triage-automation.yml

@@ -1188,7 +1188,11 @@ jobs:
               if (likelySupportedByCheckbox) reasons.push('requester confirmed Linux support via the form checkbox; no contradicting evidence found');
               if (likelySupportedByCheckbox) reasons.push('requester confirmed Linux support via the form checkbox; no contradicting evidence found');
 
 
               let verdictLine = 'Linux support could not be confirmed automatically from the submitted details.';
               let verdictLine = 'Linux support could not be confirmed automatically from the submitted details.';
-              if (confirmedNoLinux) {
+              if (linuxLabelOverrideBy && shouldApplyNoLinuxLabel) {
+                verdictLine = 'A maintainer has confirmed this server has no native Linux support.';
+              } else if (linuxLabelOverrideBy && shouldApplyConfirmedLinuxLabel) {
+                verdictLine = 'A maintainer has confirmed native Linux support.';
+              } else if (confirmedNoLinux) {
                 verdictLine = 'This server request does **not** appear to have native Linux support, which is required for LinuxGSM.';
                 verdictLine = 'This server request does **not** appear to have native Linux support, which is required for LinuxGSM.';
               } else if (suggestsNoLinux) {
               } else if (suggestsNoLinux) {
                 verdictLine = 'This server request **may not** have native Linux support based on submitted evidence.';
                 verdictLine = 'This server request **may not** have native Linux support based on submitted evidence.';