-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Use autosectionlabel extension #4146
Use autosectionlabel extension #4146
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like these changes. I think it's way more clear to know where we are linking to.
I would like to know what happen if we change the title of one of those linked sections. Do we get a lint error? Do we end up with a broken link?
We end with a |
OK. I'm interested in making the errors (as changing the title of a section) explicit and communicate them to us. Otherwise, we will end with broken links without knowing it. |
We could also make the travis-ci |
@ericholscher I checked that yesterday, we only have one warning about the http://www.sphinx-doc.org/en/master/usage/configuration.html#confval-nitpicky |
Huh, interesting. |
So it looks like our |
I actually currently don't see our default Sphinx output in the log for docs locally! |
This discussion moved to #4166 |
I just did a grep to find those links and also to make sure they aren't used in other parts. There are still some links that are used externally (like the yaml file I think) or are out of the scope like the changelog (but I think we can still have those links, going to investigate later).
Closes #4140