Skip to content

Commit 4024878

Browse files
authored
Merge pull request #22357 from github/repo-sync
repo sync
2 parents 57e7c67 + 0597111 commit 4024878

File tree

1 file changed

+4
-10
lines changed

1 file changed

+4
-10
lines changed

.github/actions/rendered-content-link-checker.js

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -430,7 +430,7 @@ async function commentOnPR(core, octokit, flaws, opts) {
430430
}
431431
}
432432

433-
function flawIssueDisplay(flaws, opts, includeExternalLinkList = true) {
433+
function flawIssueDisplay(flaws, opts, mentionExternalExclusionList = true) {
434434
let output = ''
435435
let flawsToDisplay = 0
436436

@@ -488,16 +488,10 @@ function flawIssueDisplay(flaws, opts, includeExternalLinkList = true) {
488488
}
489489
}
490490

491-
if (includeExternalLinkList) {
491+
if (mentionExternalExclusionList) {
492492
output +=
493-
'\n\n## External URLs\n\nThe following external URLs must be verified manually. If an external URL gives a false negative, add it to the file `lib/excluded-links.js`\n\n'
494-
for (const link of excludedLinks) {
495-
if (typeof link === 'string') {
496-
output += `\n - [${link}](${link})`
497-
} else {
498-
output += `\n - Pattern: \`${link.toString()}\``
499-
}
500-
}
493+
'\n\n---\n\nIf any link reported in this issue is not actually broken ' +
494+
'and repeatedly shows up on reports, consider making a PR that adds it as an exception to `lib/excluded-link.js`.'
501495
}
502496

503497
return `${flawsToDisplay} broken${

0 commit comments

Comments
 (0)