Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Check and update external URLs #11

Merged
merged 5 commits into from
Jan 10, 2024
Merged

Check and update external URLs #11

merged 5 commits into from
Jan 10, 2024

Conversation

lochhh
Copy link
Collaborator

@lochhh lochhh commented Dec 7, 2023 •
edited
Loading

This PR concerns external URLs and

  • fixes broken or redirected URLs in the documentation (except for the "all-version" Zenodo DOI URL, which is expected to resolve to the latest version) and resolves Fix broken links and redirects #7 .
  • adds the Sphinx linkcheck step in the docs build workflow to verify URLs.
  • creates aliases to base URLs of websites referenced more than once by collecting these under myst_url_schemes in conf.py.
  • adds linkcheck instructions to readme.md

@lochhh lochhh marked this pull request as ready for review December 8, 2023 13:43
@lochhh lochhh requested a review from jkbhagatio December 8, 2023 13:52
@lochhh lochhh changed the title Fix broken external URLs Check external URLs Dec 11, 2023
@lochhh lochhh changed the title Check external URLs Check and update external URLs Dec 11, 2023
@lochhh lochhh mentioned this pull request Dec 20, 2023
Merged
1 task
@jkbhagatio jkbhagatio merged commit 9174f43 into SainsburyWellcomeCentre:main Jan 10, 2024
2 checks passed
@lochhh lochhh deleted the fix-external-urls branch January 30, 2024 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jkbhagatio jkbhagatio jkbhagatio approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix broken links and redirects
2 participants
@lochhh @jkbhagatio