-
Notifications
You must be signed in to change notification settings - Fork 654
Mode and configuration documentation cleanup #989
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
Mode and configuration documentation cleanup #989
Conversation
c07178e
to
7fe140d
Compare
@asbjornu @JakeGinnivan is there a way to run readthedocs locally to test how it works before pushing? |
Yeah, install mkdocs locally, then run |
7fe140d
to
16997a6
Compare
@JakeGinnivan Thanks for the tip. I clicked through the documentation locally and found a bug plus a better way to organize the versioning modes. I've also rebased the PR. |
@JakeGinnivan thanks! I have it up and running locally now as well (after a little fighting with python and pip) |
Changed the configuration options list items into headers, so each section is easier to read and write and can fit examples, sub-sections and such.
ebf1fa6
to
9cf8c30
Compare
@JakeGinnivan I added a test in 9cf8c30 to ensure that all documentation files exist in the index. Through that, I found one that wasn't ( |
@asbjornu that is awesome! |
This PR should fix a dead link from
configuration.md
toversioning-mode.md
, moves the latter file into thereference
folder, consolidates its content with the different mode files and cleans up all related files in many various ways that I hope should improve this part of the documentation. I have no idea how this looks and works on ReadTheDocs, but fingers crossed this doesn't break anything!