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.

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

change internal links in bioconda posts #43

Merged
merged 4 commits into from
Sep 18, 2024
Merged

change internal links in bioconda posts #43

merged 4 commits into from
Sep 18, 2024

Conversation

pmenzel
Copy link
Contributor

@pmenzel pmenzel commented Sep 17, 2024

change the internal links in the bioconda posts so that they generate URLs with .html extension

@pmenzel pmenzel requested a review from a team as a code owner September 17, 2024 19:57
@pmenzel pmenzel requested a review from hexylena September 17, 2024 19:58
Copy link
Member

@hexylena hexylena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TIL about post_url that's very convenient!

I tested this locally and realised that checking by hand was a bit silly when we have automated means :) So I ran lychee locally and found a couple missed links (4fe3aab) and a bad image url (81d0aea) so I've added it to our pull request build process, not only do we check if the site builds, but we also check that every internal link (including e.g. opengraph metadata) works on every PR.

I don't think contributors need to run lychee locally as it can just run in github actions, it can more or less be ignored until a build fails because of a bad link.

@hexylena hexylena merged commit 90e3a0c into main Sep 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants