8000 Fix remaining links in docs by dstansby · Pull Request #13861 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]

Skip to content

Fix remaining links in docs #13861

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
Apr 3, 2019
Merged

Fix remaining links in docs #13861

merged 1 commit into from
Apr 3, 2019

Conversation

dstansby
Copy link
Member
@dstansby dstansby commented Apr 3, 2019

This should be most of the rest of the broken links fixed.

@dstansby dstansby added this to the v3.1.0 milestone Apr 3, 2019
@@ -901,7 +901,7 @@ Some helpful functions::

.. _ReST: http://docutils.sourceforge.net/rst.html
.. _Sphinx: http://www.sphinx-doc.org
.. _documentation: http://www.sphinx-doc.org/contents.html
.. _documentation: https://www.sphinx-doc.org/en/master/contents.html
Copy link
Member

Choose a reason for hiding this comment

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

Maybe

Suggested change
.. _documentation: https://www.sphinx-doc.org/en/master/contents.html
.. _documentation: https://www.sphinx-doc.org/en/stable/contents.html

Copy link
Member Author

Choose a reason for hiding this comment

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

I wondered about that, but https://www.sphinx-doc.org redirects to the master version

@timhoffm timhoffm merged commit 5879905 into matplotlib:master Apr 3, 2019
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Apr 3, 2019
@dstansby dstansby deleted the linkdocs branch April 3, 2019 15:27
dstansby added a commit that referenced this pull request Apr 3, 2019
…861-on-v3.1.x

Backport PR #13861 on branch v3.1.x (Fix remaining links in docs)
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
0