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 support for Travis CI #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add support for Travis CI #5

wants to merge 1 commit into from

Conversation

Muffo
Copy link

@Muffo Muffo commented Jan 29, 2018

The script:

  • Verifies that the the pdf can be generated without errors
  • Publishes the latest version to Github pages

In order for this to work, Travis CI must be configured for this project.
You will also need to generate an authentication token for Github so that Travis can publish the latest version of the pdf.

This page has all the instructions but I am happy to help if they are not clear :)

The link to the pdf has also been added to the README so that people can see a preview of the book without installing Tex Live on their system.
It won't work until Travis has been configured, but in the meantime you can see a preview of the result here: https://muffo.github.io/openssl-book/openssl-book.pdf

README Outdated
@@ -23,6 +23,9 @@ contain content that should enable the reader to get to a level of familiarity
with OpenSSL that they can then use the other documentation sources (such as the
man pages) to find detailed information as required.

The latest version of the book is available at:
Copy link
Member

Choose a reason for hiding this comment

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

I'd change "version" to "draft" ;-)

Copy link
Author

Choose a reason for hiding this comment

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

Good point! 👍

- Verify that the the pdf can be generated without errors
- Publish the latest version to Github pages

The link to the pdf has also been added to the README.
Copy link
Member

@mattcaswell mattcaswell left a comment

Choose a reason for hiding this comment

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

Awesome!

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.

3 participants