Skip to content

DOC: Fix toc structure in explain/interactive #24187

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 2 commits into from
Oct 16, 2022

Conversation

timhoffm
Copy link
Member

@timhoffm timhoffm commented Oct 16, 2022

PR Summary

Closes #22905. Not sure why we had the .. toctree:: entry there or what sphinx is doing with it. But it's not needed and without it the page becomes a structure leaf like it's sibling pages.

@timhoffm timhoffm added this to the v3.6-doc milestone Oct 16, 2022
@tacaswell
Copy link
Member

It is failing in the docs build, but it looks like we merged some bad markup someplace else?

@timhoffm
Copy link
Member Author

The doc failure was introduced in #24157 due to incorrect ReST formatting.

For simplicity, I've added a second commit with the fix here.

@timhoffm
Copy link
Member Author

I'll self-merge based on @tacaswell's positive review.

@timhoffm timhoffm merged commit 23730b4 into matplotlib:main Oct 16, 2022
@timhoffm timhoffm deleted the doc-interactive branch October 16, 2022 20:36
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Oct 16, 2022
@lumberbot-app
Copy link

lumberbot-app bot commented Oct 16, 2022

Something went wrong ... Please have a look at my logs.

It seems that the branch you are trying to backport to does not exist.

oscargus added a commit that referenced this pull request Oct 16, 2022
…187-on-v3.6.x

Backport PR #24187 on branch v3.6.x (DOC: Fix toc structure in explain/interactive)
@QuLogic QuLogic modified the milestones: v3.6-doc, v3.6.2 Oct 17, 2022
@ksunden ksunden mentioned this pull request Feb 20, 2023
6 tasks
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.

[Doc]: Duplicated toc entries
3 participants