FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

DOC: Remove explicit link to source file for Sphinx 1.5 compat. by QuLogic · Pull Request #7629 · matplotlib/matplotlib · GitHub

Repository navigation

DOC: Remove explicit link to source file for Sphinx 1.5 compat. - #7629

Merged
tacaswell merged 1 commit into
matplotlib:v2.xfrom
QuLogic:sphinx-151
Dec 15, 2016
Merged

tacaswell merged 1 commit into
matplotlib:v2.xfrom
QuLogic:sphinx-151

Conversation

QuLogic commented Dec 15, 2016

Copy link
Copy Markdown
Member

As this breaks linkchecker, I would like to self-merge to get Travis working again.

QuLogic added this to the 2.0 (style change major release) milestone Dec 15, 2016

QuLogic commented Dec 15, 2016 •
edited
Loading

Copy link
Copy Markdown
Member Author

Oops, I misread what it was looking for; with 1.5.1, it creates source as <fullname>.<ext>, but in previous versions it did <basename>.<ext>. We can configure <ext>, but as far as I can tell, not whether it uses the basename or the fullname.

The faq/howto_faq.rst is the only explicit link to the sources we have; maybe we ought to just remove it?

Copy link
Copy Markdown
Member

👍 to just remove the link.

In Sphinx 1.5, the naming of the reST source files has changed, so it's
not possible to make an explicit link that works across versions. And
users can just follow the "Show Source" link mentioned in the text
anyway.
QuLogic changed the title DOC: Remove extra .txt for sources in Sphinx 1.5. DOC: Remove explicit link to source file for Sphinx 1.5 compat. Dec 15, 2016
tacaswell merged commit 633add3 into matplotlib:v2.x Dec 15, 2016
QuLogic deleted the sphinx-151 branch December 15, 2016 22:08
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL