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 CI config to release built HTML book on tag #37

Merged
merged 10 commits into from Aug 8, 2016

Conversation

louy2
Copy link
Contributor

@louy2 louy2 commented Jul 31, 2016

I think it is convenient to have an offline version of the book for download as well, so I wrote this Travis CI config to do that. With it Travis CI can take care of releasing a zip archive of the HTML book on tag. Creating a release on Github also triggers it to append the archive to the release.

Of course, you need to go to Travis CI to enable it for this repo. It also requires env var GH_DEPLOY_TOKEN be filled with a Github personal access token with "public_repo" enabled. This can be set in the Settings tab in the Travis CI page of this repo.

@louy2
Copy link
Contributor Author

louy2 commented Jul 31, 2016

An example build.

@edunham
Copy link
Member

edunham commented Aug 2, 2016

@louy2 Thank you!

Following https://twitter.com/Gankro/status/759861678863372288, I'm currently working with the various rust-* GitHub organizations to see what org it would be most appropriate to maintain this repo in long-term. Once it lands in an org (hopefully within the next day or two), I'll set up Travis and get your changes merged!

@louy2
Copy link
Contributor Author

louy2 commented Aug 3, 2016

@edunham Thanks for taking up the maintenance!

I need to change the script a bit to zip the upper folder in as well, otherwise code highlighting doesn't work and redundant code can be seen. So please wait a bit till then to merge.

@edunham
Copy link
Member

edunham commented Aug 3, 2016

@louy2 And we've landed! Travis is now enabled on this repo.

I'd like to land your .travis.yml pretty soon so that we can start adding lints without conflicting against this PR, and actually testing PRs :)

@Manishearth Manishearth mentioned this pull request Aug 3, 2016
@louy2
Copy link
Contributor Author

louy2 commented Aug 5, 2016

I am sorry for the delay! The released zip now contains the necessary rust.css to be displayed normally. It is ready for merge.

@edunham edunham merged commit 29ef30a into rust-unofficial:master Aug 8, 2016
@edunham
Copy link
Member

edunham commented Aug 8, 2016

Thank you so much for all the work on this!

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

2 participants