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 htmlproofer via Travis! #17

Merged
merged 1 commit into from Sep 29, 2016
Merged

Add htmlproofer via Travis! #17

merged 1 commit into from Sep 29, 2016

Conversation

davetcoleman
Copy link
Member

Also fix broken links so the new tests pass

It would be great if someone removed the wiki links in the future, and transferred all the content from the wiki to these tutorials. For now though I've re-hosted the wiki temporarily so that the build passes.

This PR took a long time to implement, please do not ask me to fix all the wiki links, my temporary solution is better than 404 errors!

@130s
Copy link
Contributor

130s commented Sep 29, 2016

+1 for adding .travis.yml.
Job is failing at ruby again though.

Fix broken links
@davetcoleman
Copy link
Member Author

@mintar I could use some help figuring out how to install all the right ruby dependencies (ive never used ruby before) on Travis. I got it to work on 16.04 locally and on 14.04 on Docker, but Travis has a different environment and html-proofer won't build correctly

@davetcoleman
Copy link
Member Author

nevermind, this actually just passed finally! ready to merge.

@davetcoleman davetcoleman mentioned this pull request Sep 29, 2016
@v4hn v4hn merged commit 87bc2bd into moveit:master Sep 29, 2016
@davetcoleman davetcoleman deleted the travis branch September 29, 2016 17:43
davetcoleman pushed a commit to davetcoleman/moveit_tutorials that referenced this pull request Apr 13, 2018
* moveit commander

* removing ref to pr2 and minor cleanup to moveit_commander tutorial

* changing bold to code
davetcoleman pushed a commit that referenced this pull request May 15, 2018
* moveit commander

* removing ref to pr2 and minor cleanup to moveit_commander tutorial

* changing bold to code
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.

None yet

3 participants