Skip to content

⚡ Use warnings as errors in python api docs build#46

Open
simonrainerson wants to merge 1 commit into
mainfrom
python-strict-docs
Open

⚡ Use warnings as errors in python api docs build#46
simonrainerson wants to merge 1 commit into
mainfrom
python-strict-docs

Conversation

@simonrainerson
Copy link
Copy Markdown
Member

This makes the build of the docs fail on warnings, there is a -n
(--nitpicky) flag as well but from the description this seems a bit
overkill. This combination of short and long format flags seems to work
best across many versions of sphinx.

--keep-going (default in "newer" versions) makes it keep going so it
produces all the warnings before exiting with 1, instead of failing
on the first one.

Copy link
Copy Markdown
Member

@rasuttmus rasuttmus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Broken docs has annoyed me for quite some time 😸

Comment thread CHANGELOG.md
Comment on lines +12 to +13
- python: Building API docs now runs with warnings as errors, can be
disabled with strictDocsBuild=false.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Guess this should go under "Unreleased"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

now it is :)

This makes the build of the docs fail on warnings, there is a -n
(--nitpicky) flag as well but from the description this seems a bit
overkill. This combination of short and long format flags seems to work
best across many versions of sphinx.

--keep-going (default in "newer" versions) makes it keep going so it
produces all the warnings before exiting with 1, instead of failing
on the first one.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants