Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add simple travis-ci support #25

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

Closed
wants to merge 2 commits into from
Closed

Conversation

kragniz
Copy link
Contributor

@kragniz kragniz commented Jul 24, 2016

This adds a simple travis file that builds the docs, failing if warnings occur. There were a few broken references causing errors, so they were also removed. Closes #3.

@@ -218,11 +218,11 @@ grasp a simple example more quickly than they can digest a formal description in
prose.

People learn faster with concrete, motivating examples that match the context of
a typical use case. For instance, the :func:`str.rpartition` method is better
a typical use case. For instance, the ``str.rpartition`` method is better
Copy link
Member

Choose a reason for hiding this comment

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

The intersphinx_mapping setting in https://github.com/python/devguide/blob/master/conf.py#L33 should resolve this and sys.ps2. Perhaps Sphinx didn't follow http -> https redirection?

@brettcannon
Copy link
Member

I have turned on Travis so that next time this PR gets updated to address my and @berkerpeksag 's comment it should run the check.

@kragniz
Copy link
Contributor Author

kragniz commented Aug 6, 2016

Thanks! Currently at EMF camp, will address the comments later this week

@brettcannon
Copy link
Member

Any chance of getting the changes addressed, @kragniz ?

@brettcannon brettcannon mentioned this pull request Oct 21, 2016
@brettcannon
Copy link
Member

Thanks for the initial attempt, @kragniz , but the need to have this support meant I forked this PR and took it the rest of the way in #69 .

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

Successfully merging this pull request may close these issues.

4 participants