Skip to content
This repository has been archived by the owner on Jul 5, 2021. It is now read-only.

Hotfix: Add tests to check for broken links #13

Closed
dylanPowers opened this issue Apr 24, 2015 · 8 comments
Closed

Hotfix: Add tests to check for broken links #13

dylanPowers opened this issue Apr 24, 2015 · 8 comments
Assignees
Labels
dif/medium Prior experience is likely helpful help wanted Seeking public contribution on this issue topic/design-front-end Front-end implementation of UX/UI work topic/docs Documentation

Comments

@dylanPowers
Copy link
Contributor

It would be good to have a tool that can test for broken links and give us a little piece of mind that we aren't frustrating anyone out there.

Tools:

@dylanPowers
Copy link
Contributor Author

http://wummel.github.io/linkchecker/ works really well. The only disadvantage is that it doesn't parse the fragment links and see if the html id tags exist. We should integrate it into the build to be ran before deployment.

@harlantwood
Copy link

Maybe it could work on CI...

@dylanPowers
Copy link
Contributor Author

We haven't set up CI for this yet, but that's a good idea.

@jbenet
Copy link
Contributor

jbenet commented Jun 6, 2015

+1

This was referenced Jun 6, 2015
@whyrusleeping whyrusleeping mentioned this issue Jun 9, 2015
50 tasks
@jbenet jbenet mentioned this issue Jun 16, 2015
55 tasks
@dignifiedquire dignifiedquire added enhancement help wanted Seeking public contribution on this issue and removed codereview labels Jul 26, 2016
@dignifiedquire dignifiedquire changed the title Tool that tests for broken links needed Add tests to check for broken links Jul 26, 2016
@meiqimichelle
Copy link
Contributor

This may also work (haven't tried it, just happened to see it): https://github.com/peaceiris/actions-hugo-link-check

@jessicaschilling jessicaschilling changed the title Add tests to check for broken links Hotfix: Add tests to check for broken links Aug 26, 2019
@jessicaschilling jessicaschilling added topic/design-front-end Front-end implementation of UX/UI work dif/medium Prior experience is likely helpful topic/docs Documentation OKR: Hotfixes and removed P3 - Low labels Aug 26, 2019
@jessicaschilling
Copy link
Contributor

@cwaring -- Discovered this when grooming stale issues. Can we please check in about the best way to get it included in future efforts? Seems it manifests/would manifest in two ways:

  1. for main ipfs.io site, ideally in CI (do we already do this, actually?)
  2. for revamped docs site (is this already in your requirements list?)

@cwaring
Copy link
Contributor

cwaring commented Aug 28, 2019

It would be great to have this at CI level for immediate alerts, however now that we have Google Search Console enabled we should receive alerts for any 404s which will also help us take appropriate care with any inbound redirects that we may have missed.

A blend of the two methods would be ideal for full coverage but I'm confident that the GSC provides robust coverage for now - especially if we emit a sitemap.xml with every new update.

@jessicaschilling
Copy link
Contributor

@cwaring - Thank you, makes sense. Closing this issue with the caveat that we remain aware of this as we revamp overall docs platform.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dif/medium Prior experience is likely helpful help wanted Seeking public contribution on this issue topic/design-front-end Front-end implementation of UX/UI work topic/docs Documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants