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

try rtfd #3826

Merged
merged 1 commit into from Jul 29, 2013
Merged

try rtfd #3826

merged 1 commit into from Jul 29, 2013

Conversation

Carreau
Copy link
Member

@Carreau Carreau commented Jul 29, 2013

Fix rtfd build.

@@ -32,7 +32,8 @@

# Import support for ipython console session syntax highlighting
# (lives IPython's sphinxext subpackage)
from IPython.sphinxext import ipython_console_highlighting
if not ON_RTD:
from IPython.sphinxext import ipython_console_highlighting
Copy link
Member

Choose a reason for hiding this comment

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

Does this need to be imported at all?

Copy link
Member Author

Choose a reason for hiding this comment

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

You added it not so long ago, so I supposed it was.

@Carreau
Copy link
Member Author

Carreau commented Jul 29, 2013

fix, rebased and squashed.

@ivanov
Copy link
Member

ivanov commented Jul 29, 2013

looks good to merge, though I don't actually see any failures on ReadTheDocs...

minrk added a commit that referenced this pull request Jul 29, 2013
@minrk minrk merged commit c5e2b15 into ipython:master Jul 29, 2013
@Carreau
Copy link
Member Author

Carreau commented Jul 29, 2013

@ivanov rtfd is not configured to build master by default. You are building the repo as it was 4 month ago. WIll change now that I am admin.

mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
@Carreau Carreau deleted the fixrtfd branch December 15, 2014 16:47
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