Skip to content

Conversation

@sbryngelson
Copy link
Member

@sbryngelson sbryngelson commented Nov 14, 2024

This is an attempt to try to fix #671 and #716 at the same time

This PR moves the link checking to the HTML files created by the documentation build/CI.

Lychee is now used to check for dead links.

It works, and finds dead links that have been around a long time. It is currently configured to pass CI even if it finds dead links, but once the dead links are fixed, we will change this.

@codecov
Copy link

codecov bot commented Nov 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 42.96%. Comparing base (e0317d7) to head (c1c1ce5).
Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #733   +/-   ##
=======================================
  Coverage   42.96%   42.96%           
=======================================
  Files          61       61           
  Lines       16339    16339           
  Branches     1881     1881           
=======================================
  Hits         7020     7020           
  Misses       8278     8278           
  Partials     1041     1041           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sbryngelson
Copy link
Member Author

sbryngelson commented Nov 14, 2024

this isn't right. need to switch to something that checks HTML not markdown. trying what I
discuss here #671

update: fixed.

@sbryngelson sbryngelson marked this pull request as draft November 14, 2024 17:52
@sbryngelson sbryngelson marked this pull request as ready for review November 15, 2024 15:09
@sbryngelson sbryngelson added continuous-integration Continuous integration (CI) website Website changes labels Nov 15, 2024
@sbryngelson sbryngelson linked an issue Nov 15, 2024 that may be closed by this pull request
@sbryngelson sbryngelson changed the title Try a new link inspector for website/docs Switch to checking for deadlinks in the docs/CI HTML via Lychee Nov 15, 2024
@sbryngelson sbryngelson changed the title Switch to checking for deadlinks in the docs/CI HTML via Lychee Switch to checking for dead links in the docs/CI HTML via Lychee Nov 15, 2024
@sbryngelson sbryngelson merged commit ec01950 into MFlowCode:master Nov 15, 2024
14 checks passed
@sbryngelson sbryngelson deleted the linkspec branch November 16, 2024 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

continuous-integration Continuous integration (CI) website Website changes

Development

Successfully merging this pull request may close these issues.

Move link checker to linkspector LinkChecker validation inconsistent between Markdown and HTML

1 participant