-
Notifications
You must be signed in to change notification settings - Fork 343
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
Sphinx warnings and errors fix #5815
Conversation
Dear contributor,
As for the Avocado utility modules (“avocado.utils”) it is OK to introduce new functionality, |
5a08f3c
to
06be3ae
Compare
Reference: avocado-framework#5757 Signed-off-by: Jan Richter <[email protected]>
Signed-off-by: Jan Richter <[email protected]>
The sphinx-apidoc command didn't respect the namespaces of optional plugins, because of that the generated documentation didn't point to right modules. Signed-off-by: Jan Richter <[email protected]>
Signed-off-by: Jan Richter <[email protected]>
varianter plugins doc has been moved to its own subsection therefore we need to remove them from toctree. Signed-off-by: Jan Richter <[email protected]>
Signed-off-by: Jan Richter <[email protected]>
Signed-off-by: Jan Richter <[email protected]>
Signed-off-by: Jan Richter <[email protected]>
Signed-off-by: Jan Richter <[email protected]>
Signed-off-by: Jan Richter <[email protected]>
Signed-off-by: Jan Richter <[email protected]>
Signed-off-by: Jan Richter <[email protected]>
This commit introduces the static checks for avocado documentation. This change will help us avoid documentation issues in the future. It makes the readthedocs build more strict, which make it fail on any warning. Reference: avocado-framework#5757 Signed-off-by: Jan Richter <[email protected]>
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.
LGTM, thanks!
The one RPM build stuck is certainly not related to these changes, so merging AS IS.
This PR fixes Sphinx warnings and errors in our documentation and adds static tests to avoid those problems in the future.
Reference: #5757
Signed-off-by: Jan Richter [email protected]