The new lines weren't working properly as we'd filter them out, whoops (#1951)

This commit is contained in:
Aaron Powell
2026-06-09 21:44:16 -07:00
committed by GitHub
parent d93e8cf931
commit 50854076f2
3 changed files with 5 additions and 5 deletions
@@ -233,7 +233,7 @@ jobs:
...unmatchedRows
].join('\n')
: '',
].filter(Boolean).join('\n');
].join('\n');
if (existingTrackerIssues.length > 0) {
const [primary, ...duplicates] = existingTrackerIssues;
+1 -1
View File
@@ -214,7 +214,7 @@ jobs:
exitCode !== '0'
? '> **Note:** The validator returned a non-zero exit code. Please review the findings above before merge.'
: '',
].filter(Boolean).join('\n');
].join('\n');
// Find existing comment with our marker
const { data: comments } = await github.rest.issues.listComments({
+3 -3
View File
@@ -553,7 +553,7 @@ function buildMergedIntakeComment(baseResult, qualityResult, runId, owner, repo)
? ["", "### Warnings", "", ...baseResult.warnings.map((warning) => `- ${warning}`)].join("\n")
: "",
runLink ? `\n${runLink}` : "",
].filter(Boolean).join("\n");
].join("\n");
}
export function applyQualityGateResult(baseEvaluation, qualityGateResult, runId, owner, repo) {
@@ -644,7 +644,7 @@ export async function evaluateExternalPluginIssue({ issue, token, runId, owner,
? ["", "### Warnings", "", ...dedupedWarnings.map((warning) => `- ${warning}`)].join("\n")
: "",
runLink ? `\n${runLink}` : "",
].filter(Boolean).join("\n")
].join("\n")
: [
marker,
"## ❌ External plugin intake failed",
@@ -659,7 +659,7 @@ export async function evaluateExternalPluginIssue({ issue, token, runId, owner,
? ["", "### Warnings", "", ...dedupedWarnings.map((warning) => `- ${warning}`)].join("\n")
: "",
runLink ? `\n${runLink}` : "",
].filter(Boolean).join("\n");
].join("\n");
return {
valid,