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

Skip to content

DOC: add SOURCELINK_SUFFIX for compatibility with Sphinx 1.5 #8086

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
Feb 17, 2017

Conversation

mitya57
Copy link
Contributor

@mitya57 mitya57 commented Feb 16, 2017

With Sphinx 1.5, this is needed by searchtools.js to display the source snippets (see sphinx-doc/sphinx#2454, and especially sphinx-doc/sphinx@71dd8bfbf).

With earlier Sphinx versions, this is a no-op because the undefined variable will evaluate to an empty string.

With Sphinx 1.5, this is needed by searchtools.js to display the source
snippets (see sphinx-doc/sphinx#2454).

With earlier Sphinx versions, this is a no-op because the undefined variable
will evaluate to an empty string.
@NelleV
Copy link
Member

NelleV commented Feb 16, 2017

wow, that is an annoying change on the behalf of sphinx… It should be taken care of out of the box.

@NelleV NelleV changed the title DOC: add SOURCELINK_SUFFIX for compatibility with Sphinx 1.5 [MRG+1] DOC: add SOURCELINK_SUFFIX for compatibility with Sphinx 1.5 Feb 16, 2017
Copy link
Member

@NelleV NelleV left a comment

Choose a reason for hiding this comment

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

👍

Thanks @mitya57 !

@tacaswell tacaswell merged commit de72808 into matplotlib:master Feb 17, 2017
tacaswell added a commit that referenced this pull request Feb 17, 2017
DOC: add SOURCELINK_SUFFIX for compatibility with Sphinx 1.5
@tacaswell
Copy link
Member

Thanks, backported to v2.0.0-doc as 1cce7d3

@tacaswell tacaswell added this to the 2.0.1 (next bug fix release) milestone Feb 17, 2017
@QuLogic QuLogic changed the title [MRG+1] DOC: add SOURCELINK_SUFFIX for compatibility with Sphinx 1.5 DOC: add SOURCELINK_SUFFIX for compatibility with Sphinx 1.5 Feb 17, 2017
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