-
Notifications
You must be signed in to change notification settings - Fork 8
Fix link checker #113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix link checker #113
Changes from all commits
1ec35ee
fbd5387
62d639b
e5dd665
9893b3b
16a6b93
d2e9a7b
fb22f1a
3e890bb
f4f6da2
e00c97f
a6d925d
c287f94
6fa9283
3501f30
858dbdc
b7e142a
e79c6b9
26ec8ed
5813805
7e8adb6
d496aa7
0729dff
2c27902
0ecd442
adb5f7f
4ef29f5
59d6217
7d03e74
8bfcfce
cfb82d7
bfba267
d26b05a
8b2b2a5
2abbb27
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -27,5 +27,5 @@ | |
| title: Open Research Leadership Development - Preparing researchers to nurture healthy communities. | ||
| link: | ||
| funding: UCL Research Culture Seed Funds | ||
| link_call: https://www.ucl.ac.uk/research/strategy-environment/research-culture-ucl/get-involved/research-culture-seed-fund | ||
| link_call: | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
| result: ❌ | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,10 +1,30 @@ | ||
| accept = [200, 403, 415, 429] | ||
| accept = [200, 202, 403, 415, 429, 999] | ||
| verbose = "info" | ||
| no_progress = true | ||
| exclude = [ | ||
| 'https://www.softwareheritage.org', # Their SSL certificate is not trusted but the link works (currently!) | ||
| '(?x).*profiles.ucl.ac.uk.*/thumbnail', # People may not have a profile image. | ||
| '(?x).*github-readme-stats.vercel.app.*', # Returns 503 for some reason | ||
| '(?x).rdr.ucl.ac.uk/articles/*', # Article does exist but sometimes returns strange codes: 202, 403... | ||
| '(?x).*127.0.0.1.*', # This is a local address | ||
| '(?x).*liveuclac.sharepoint.com/sites/*', # SharePoint pages will need UCL credentials. | ||
| # Several annoying false positives: | ||
| # | ||
| # Some parts of the upstream theme cause errors because they don't prepend https. We don't fix the theme because | ||
| # it will be updated to the new branding soon. So just suppress all of this for now. Remove when the theme is | ||
| # updated. All social media links on every theme-rendered page are like this!! | ||
|
Comment on lines
+13
to
+15
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It is annoying the lychee seems to complain about so many URLs
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes. Yes it is. Also the theme is starting to work against us. |
||
| '(?x).*cdn.ucl.ac.uk/*', | ||
| '(?x)file://(?:.*/)?www\.ucl\.ac\.uk(?:/|$)', | ||
| '(?x).*shop.ucl.ac.uk', | ||
| '(?x).*www.flickr.com/photos/uclnews', | ||
| '(?x).*www.facebook.com/uclofficial', | ||
| '(?x).*twitter.com/ucl', | ||
| '(?x).*www.instagram.com/ucl', | ||
| '(?x).*www.youtube.com/ucltv', | ||
| '(?x).*soundcloud.com/uclsound', | ||
| # Aaand the favicon and several image paths are broken in the upstream theme, so just ignore these too. | ||
| '(?x).*favicon-152.png', | ||
| '(?x).*assets/images/hero-image-taught-desktop.jpg', # doesn't exist: theme bug | ||
| '(?x).*indigo/images/search-sprite.png', # doesn't exist: theme bug | ||
| '(?x).*assets/css/owl.video.play.png', | ||
| ] | ||

Uh oh!
There was an error while loading. Please reload this page.