Skip to content

ENH: use lychee for linkchecking #349

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

Merged
merged 10 commits into from
Jun 6, 2025
Merged

ENH: use lychee for linkchecking #349

merged 10 commits into from
Jun 6, 2025

Conversation

mmcky
Copy link
Contributor

@mmcky mmcky commented Jun 5, 2025

This PR sets up lychee for more robust link checking and:

  1. removes link-checking from the PR process (as a distraction)
  2. uses lychee as a more robust solution to parse html
  3. checks the live site gh-pages every day.
  4. Posts an issue with any results (excluding 403 and 503)

Interestingly the only 503 results are all the AI company websites. They are blocking web traffic requests! Kind of ironic given how they have been trained.

This PR also fixes the canonical site address from python. to python-programming. that is embedded in the html.


Issues with checking md directly.

These types of links [lecture on broadcasting](broadcasting) get resolved to a web address by default which produces an error.

  • perhaps should use html parsing instead of md parsing to avoid some of these minor syntax issues that Jupyter-book resolves at build time.

Copy link

github-actions bot commented Jun 5, 2025

@mmcky mmcky closed this Jun 5, 2025
@mmcky mmcky reopened this Jun 5, 2025
@mmcky mmcky closed this Jun 5, 2025
@mmcky mmcky reopened this Jun 5, 2025
@mmcky mmcky closed this Jun 5, 2025
@mmcky mmcky reopened this Jun 5, 2025
@github-actions github-actions bot temporarily deployed to pull request June 5, 2025 06:27 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 5, 2025 06:28 Inactive
@mmcky mmcky closed this Jun 5, 2025
@mmcky mmcky reopened this Jun 5, 2025
@github-actions github-actions bot temporarily deployed to pull request June 5, 2025 06:37 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 5, 2025 06:41 Inactive
@mmcky mmcky closed this Jun 5, 2025
@mmcky mmcky reopened this Jun 5, 2025
@github-actions github-actions bot temporarily deployed to pull request June 5, 2025 23:00 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 5, 2025 23:03 Inactive
@mmcky mmcky closed this Jun 5, 2025
@mmcky mmcky reopened this Jun 5, 2025
@mmcky mmcky closed this Jun 5, 2025
@mmcky mmcky reopened this Jun 5, 2025
@github-actions github-actions bot temporarily deployed to pull request June 5, 2025 23:29 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 5, 2025 23:29 Inactive
@github-actions github-actions bot temporarily deployed to pull request June 6, 2025 05:10 Inactive
@mmcky mmcky merged commit d6ec7d6 into main Jun 6, 2025
5 checks passed
@mmcky mmcky deleted the lychee branch June 6, 2025 05:20
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.

1 participant