Use dirhtml builder instead of default html #90
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
My previous PR #88 added a Read the Docs config file, which inadvertently switched from
dirhtml
builder (the default without a config file) tohtml
(the default with a config file: https://docs.readthedocs.io/en/stable/config-file/v2.html#sphinx-builder). Sorry about that!For example:
dirhtml
:python -m sphinx -T -E -b dirhtml -d _build/doctrees -D language=en . $READTHEDOCS_OUTPUT/html
https://readthedocs.org/projects/pypaio/builds/20072068/
html
:python -m sphinx -T -E -b html -d _build/doctrees -D language=en . $READTHEDOCS_OUTPUT/html
https://readthedocs.org/projects/pypaio/builds/20093063/And here's someone (python/peps#3199) noticing a previously working https://www.pypa.io/en/latest/specifications/ is now 404 and the working one is https://www.pypa.io/en/latest/specifications.html
Let's explicitly switch back to
dirhtml
to ensure other existing links aren't broken. (And we can revert python/peps#3199.)