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
Much cleanup for configuration docs #674
Conversation
* Clean up a lot of copy. * Reduce usage of internal Sphinx concepts like "toctree" * Make language clearer * Fix a few factual inaccuracies * Remove noisey/placeholder/introduction to a paragraph sort of text, this just distracts readers. * Make `confval` domain role to allow for creation of config value field lists that are linkable and can be referenced. Also include type/default fields * Reduce redundancy in heading * Intersphinx to Sphinx docs * Use new role to intersphinx to Sphinx confvals * Add copy on using logo/etc. These are common questions for us for those not familiar with Sphinx. Link to appropriate docs.
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.
Nice improvement! Small comments:
|
||
:type: string | ||
|
||
Change the Google Analytics ID that is included on pages. |
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.
Maybe note that this is not actually used yet?
:type: boolean | ||
:default: ``False`` | ||
|
||
Only display the logo image, do not display the project name at the top of |
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.
Note that the version number will still be displayed?
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.
Should we document it yet? I can comment it out for now if it's effectively a noop.
Thanks for the feedback, and sorry it took a while to return to this! I edited with all of the above feedback and fixed a couple more issues on intersphinx linking. |
It's important to note that if you don't follow the same styling for your rST headers across | ||
your documents, the toctree will misbuild, and the resulting menu might not show the correct | ||
depth when it renders. | ||
Currently the left menu will build based upon any ``toctree`` directives defined |
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.
Currently the left menu will build based upon any ``toctree`` directives defined | |
Currently the left menu will build based upon any :rst:dir:`sphinx:toctree` directives defined |
LGTM with 1 comment (above) |
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.
Great changes. Going to merge this for the next release.
this just distracts readers.
confval
domain role to allow for creation of config value fieldlists that are linkable and can be referenced. Also include
type/default fields
those not familiar with Sphinx. Link to appropriate docs.