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
Live reload #2
base: main
Are you sure you want to change the base?
Live reload #2
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.
Thanks for documenting how to do auto-builds. Could you update the PR according to the review comments?
README.rst
Outdated
2. For continuous rebuilds and live-reload in the browser while editing the docs, run `sphinx-autobuild docs/source docs/build/html --open-browser` | ||
3. Build the HTML version using ``make html`` (in the ``docs`` folder). |
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.
Putting the info for continuous rebuilds in this list makes it look like this is a necessary step in order to build the docs. I think it should rather go below the enumeration.
sphinx-autobuild |
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.
Could you squash this fixup commit into the one that adds this line?
.gitignore
Outdated
@@ -8,3 +8,4 @@ __pycache__ | |||
coverage.xml | |||
.pytest_cache/ | |||
docs/build | |||
.vscode |
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 think support for particular editors should not be part of this repo. I’d recommend to add a global .gitignore instead that takes vscode into account.
@@ -57,4 +57,4 @@ | |||
# Add any paths that contain custom static files (such as style sheets) here, | |||
# relative to this directory. They are copied after the builtin static files, | |||
# so a file named "default.css" will overwrite the builtin "default.css". | |||
html_static_path = ['_static'] | |||
# html_static_path = ['_static'] |
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.
What is sphinx-autoreload
complaining about? Is it because the folder doesn’t exist (yet). Should we just add it as an empty folder to the repo instead?
This PR adds live-reload and continuous building while editing the docs
This makes it easier to preview your changes