Skip to content
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

Add Travis config to run linkchecker against the built site #163

Merged
merged 4 commits into from
Nov 23, 2017

Conversation

alext
Copy link
Contributor

@alext alext commented Nov 16, 2017

What

Run linkchecker against the built site so that we catch any broken links etc.

This fixes a couple of broken links that were found as a result.

I've also cleaned up an unused image that was missed in #159

How to review

  • Code review
  • Verify that the Travis build passes.

Who can review

Not me.

It's been necessary to pin an older version of requests due to an issue
with the version checking in linkchecker.
To build the site and run linkchecker over the result.
This was tripping up linkchecker.
The content that included this image was removed in 710a4bf.
@alext alext changed the title Add Travis config to run linkchecker against the build site Add Travis config to run linkchecker against the built site Nov 16, 2017
@samcrang samcrang self-assigned this Nov 22, 2017
@@ -18,3 +18,7 @@ six==1.10.0
tornado==4.3
webcolors==1.5
wsgiref==0.1.2

linkchecker==9.3
Copy link
Contributor

@samcrang samcrang Nov 22, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The linkchecker project currently seems to be in a weird state—supposedly the project is dead and has been picked up by a new group but they still haven't released anything on PyPI yet.

Does using this seemingly dead project seem like a problem?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're already using this for the tech docs, so I chose to use it here for consistency. I agree that it's in a wierd state, but I think we should address that across everything separately if that becomes an issue.

@samcrang samcrang merged commit 746fe8f into master Nov 23, 2017
@samcrang samcrang deleted the link_checker branch November 23, 2017 10:16
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.

2 participants