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

Checking for broken links with TravisCI #62

Open
peterjc opened this issue Apr 29, 2016 · 6 comments
Open

Checking for broken links with TravisCI #62

peterjc opened this issue Apr 29, 2016 · 6 comments

Comments

@peterjc
Copy link
Member

peterjc commented Apr 29, 2016

See #19 for fixing all the existing broken links.

Once that's done, we should add a link checker to TravisCI to keep appraised of any broken links (due to external site changes) or bad links in incoming pull request.

@JoaoRodrigues suggested following https://www.jacobtomlinson.co.uk/jekyll/2015/02/18/test-you-jekyll-blog-with-travis-ci/ which suggests this for the .travis.yml file:

language: ruby
rvm:
  - 2.1
script:
  - bundle exec jekyll build
  - bundle exec htmlproof ./_site --only-4xx --check-favicon --check-html
env:
  global:
    - NOKOGIRI_USE_SYSTEM_LIBRARIES=true # speeds up installation of html-proofer
@souravsingh
Copy link

@peterjc We can make use of linkchecker for checking broken links https://wummel.github.io/linkchecker/

@peterjc
Copy link
Member Author

peterjc commented Jul 22, 2016

@souravsingh have you tried using linkchecker under TravisCI to check local files which GitHub Pages and Jekyll will generate?

This repository seems to use it, https://github.com/mingwpy/mingwpy.github.io/blob/source/.travis.yml

There's an open issue on the link checker repository about how best to do this (or indeed, if we should do this): wummel/linkchecker#573

@souravsingh
Copy link

@peterjc Should we go with using linkchecker or using the solution @JoaoRodrigues suggested? I think the latter solution is the best.

@peterjc
Copy link
Member Author

peterjc commented Aug 22, 2016

@souravsingh You can try either.

I suggest you try installing and running the link checker on your own computer, and if you find any broken links please submit fixes.

Then you'd need to test this works via TravisCI. If you fork Biopython on GitHub, you should be able to turn on TravisCI for your fork. Once you have something working please make a pull request to the main Biopython website repository (which will run TravisCI again via the Biopython account).

@souravsingh
Copy link

@peterjc I have run the linkchecker on the website. Here is the list of dead links-https://gist.github.com/souravsingh/cfa0a363bfdad416e89dfe807078d99a

@peterjc
Copy link
Member Author

peterjc commented Aug 22, 2016

@souravsingh see #19 for dead links (and #7 for the releases).

Note there are some false positives in the linkchecker output, e.g. ConnectionError examples probably need a retry. Perhaps there are some linkchecker options you can use here, as otherwise this will not be useful for TravisCI testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants