Skip to content

Fix errors that were crashing doc build #122

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

Merged
merged 1 commit into from
Mar 2, 2015
Merged

Conversation

mwaskom
Copy link
Member

@mwaskom mwaskom commented Mar 2, 2015

GH-#115 removed surfer.io.read_label but did not change the example
that was referencing it. I'm surprised that got through Travis--
does Travis not build the docs?

While fixing this I also noticed a couple of things that had gotten
out of date and fixed an oversight from the config file removal.

GH-nipy#115 removed surfer.io.read_label but did not change the example
that was referencing it. I'm surprised that got through Travis--
does Travis not build the docs?

While fixing this I also noticed a couple of things that had gotten
out of date and fixed an oversight from the config file removal.
@larsoner
Copy link
Contributor

larsoner commented Mar 2, 2015

Travis does not build the docs, no. Feel free to open a PR for that :) It could be another worker condition.

@larsoner
Copy link
Contributor

larsoner commented Mar 2, 2015

LGTM, +1 for merge once the CIs are happy

@coveralls
Copy link

Coverage Status

Coverage increased (+0.11%) to 72.46% when pulling 8bf48a8 on mwaskom:fix_docs into 634cd6d on nipy:master.

larsoner added a commit that referenced this pull request Mar 2, 2015
Fix errors that were crashing doc build
@larsoner larsoner merged commit 8823211 into nipy:master Mar 2, 2015
@mwaskom mwaskom deleted the fix_docs branch March 4, 2015 18:58
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