Skip to content

Add lychee link checker workflow - #725

Open
InessaPawson wants to merge 5 commits into
scientific-python:mainfrom
InessaPawson:lychee
Open

InessaPawson wants to merge 5 commits into
scientific-python:mainfrom
InessaPawson:lychee

Conversation

@InessaPawson

Copy link
Copy Markdown
Member

Closes #702.

Needs a type: label, I don't have permissions to do it.

The workflow is set to fail: false for us to see what the first runs report. Once the existing broken links are fixed, I'd recommend flipping it to fail: true so broken links are treated as errors by CI.

@netlify

netlify Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for scientific-python-hugo-theme ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 6400bca
🔍 Latest deploy log https://app.netlify.com/projects/scientific-python-hugo-theme/deploys/6ac0d6f9e4291c000875078f
😎 Deploy Preview https://deploy-preview-725--scientific-python-hugo-theme.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 94
Accessibility: 97
Best Practices: 100
SEO: 91
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread .github/workflows/linkcheck.yaml Outdated
Comment thread lychee.toml Outdated
Comment thread Makefile Outdated
@InessaPawson

Copy link
Copy Markdown
Member Author

@agriyakhetarpal I checked the Netlify build log to see why CI fails here when the build works for me locally. The learn Makefile target fails because learn.scientific-python.org migrated from Hugo to MyST (scientific-python/learn.scientific-python.org#315). This will break every deploy preview until learn is dropped from the netlify-preview target. Happy to open a separate PR for that if it would be helpful.

@agriyakhetarpal

Copy link
Copy Markdown
Member

@agriyakhetarpal I checked the Netlify build log to see why CI fails here when the build works for me locally. The learn Makefile target fails because learn.scientific-python.org migrated from Hugo to MyST (scientific-python/learn.scientific-python.org#315). This will break every deploy preview until learn is dropped from the netlify-preview target. Happy to open a separate PR for that if it would be helpful.

Good point, @InessaPawson. I think we should indeed drop the learn site in a separate PR. Thanks for addressing my feedback so quickly!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use Lychee as link checker

2 participants