Skip to content

Add script &CI to check dead links #39

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

Open
liugddx opened this issue Dec 12, 2024 · 6 comments · May be fixed by #45
Open

Add script &CI to check dead links #39

liugddx opened this issue Dec 12, 2024 · 6 comments · May be fixed by #45
Assignees
Labels
good first issue Good for newcomers

Comments

@liugddx
Copy link
Contributor

liugddx commented Dec 12, 2024

Feature Request

Add script &CI to check dead links.

Which Areas Would Be Affected?

CI

Why Is the Feature Desired?

Every time PR is merged, we can ensure that there are no dead links.

Solution Proposal

If possible, provide a (brief!) solution proposal.

@ndr-brt
Copy link
Member

ndr-brt commented Dec 18, 2024

A proposal could be to use wget directly, it looks pretty straightforward:
https://www.digitalocean.com/community/tutorials/how-to-find-broken-links-on-your-website-using-wget-on-debian-7

otherwise a tool like htmltest could be used:
https://robb.sh/posts/check-links-in-hugo-with-htmltest/

@ndr-brt ndr-brt added the good first issue Good for newcomers label Dec 18, 2024
@liugddx
Copy link
Contributor Author

liugddx commented Dec 18, 2024

A proposal could be to use wget directly, it looks pretty straightforward:

https://www.digitalocean.com/community/tutorials/how-to-find-broken-links-on-your-website-using-wget-on-debian-7

otherwise a tool like htmltest could be used:

https://robb.sh/posts/check-links-in-hugo-with-htmltest/

I‘m willing to have a try.

@ndr-brt
Copy link
Member

ndr-brt commented Dec 18, 2024

Great! I assigned you on the issue

@liugddx liugddx linked a pull request Dec 19, 2024 that will close this issue
@ndr-brt
Copy link
Member

ndr-brt commented Feb 10, 2025

@liugddx are you still working on this? otherwise I will remove assignment and let somebody else to contribute in

@liugddx
Copy link
Contributor Author

liugddx commented Feb 10, 2025

@liugddx are you still working on this? otherwise I will remove assignment and let somebody else to contribute in

All right. I found a lot of dead links, and I can‘t be sure what the correct link is.

@ndr-brt
Copy link
Member

ndr-brt commented Feb 10, 2025

@liugddx are you still working on this? otherwise I will remove assignment and let somebody else to contribute in

All right. I found a lot of dead links, and I can‘t be sure what the correct link is.

This issue is only about having the CI job to detect dead links, then the activity to fix those should be made separately (because they could be a lot)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
2 participants