Skip to content

Commit b474767

Browse files
authored
Merge pull request #17499 from github/exclude-some-links
Exclude some links from the link check
2 parents 9434940 + 72f506c commit b474767

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

lib/excluded-links.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,5 +11,7 @@ module.exports = [
1111

1212
// Oneoff links that link checkers think are broken but are not.
1313
'https://haveibeenpwned.com/',
14-
'https://www.ilo.org/dyn/normlex/en/f\\?p=NORMLEXPUB:12100:0::NO::P12100_ILO_CODE:P029'
14+
'https://www.ilo.org/dyn/normlex/en/f\\?p=NORMLEXPUB:12100:0::NO::P12100_ILO_CODE:P029',
15+
'https://www.linkedin.com/company/github',
16+
'https://www.facebook.com/'
1517
]

script/check-english-links.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ const languagesToSkip = Object.keys(require('../lib/languages'))
4141
// Skip deprecated Enterprise content.
4242
// Capture the old format https://docs.github.com/enterprise/2.1/
4343
// and the new format https://docs.github.com/enterprise-server@2.19/.
44-
const enterpriseReleasesToSkip = new RegExp(`${root}.+?[/@](${deprecated.join('|')})/`)
44+
const enterpriseReleasesToSkip = new RegExp(`${root}.+?[/@](${deprecated.join('|')})(/|$)`)
4545

4646
const config = {
4747
path: program.path || englishRoot,

0 commit comments

Comments
 (0)