Skip to content
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

Merged
merged 13 commits into from
Dec 7, 2023
Merged

Conversation

richtja
Copy link
Contributor

@richtja richtja commented Nov 28, 2023

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]

@mr-avocado
Copy link

mr-avocado bot commented Nov 28, 2023

Dear contributor,
Avocado is currently under sprint #103, which is due to release an LTS (Long Term Stability) release.
Please avoid merging changes that do not fall into these categories:

  • Bug fixes
  • Usability Improvements
  • Documentation updates

As for the Avocado utility modules (“avocado.utils”) it is OK to introduce new functionality,
but changes to the existing APIs (including interface and behavior) should be avoided.
These kind of changes should wait until sprint #104.

@richtja richtja marked this pull request as draft November 28, 2023 14:36
@richtja richtja force-pushed the sphinx_fix branch 13 times, most recently from 5a08f3c to 06be3ae Compare December 5, 2023 10:06
@richtja richtja marked this pull request as ready for review December 5, 2023 10:25
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]>
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]>
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]>
Copy link
Contributor

@clebergnu clebergnu left a 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.

@clebergnu clebergnu merged commit b1c0a8e into avocado-framework:master Dec 7, 2023
63 of 64 checks passed
@richtja richtja deleted the sphinx_fix branch March 26, 2024 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants