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

Resolve some docs warnings #3322

Merged
merged 9 commits into from
Jul 18, 2024
Merged

Conversation

kurtmckee
Copy link
Contributor

Pull Request check-list

Please make sure to review and check all of these items:

  • Do tests and lints pass with this change?
  • Do the CI tests pass with this change (enable it first in your forked repo and wait for the github action build to finish)?
  • Is the new or changed code fully tested?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?
  • Is there an example added to the examples folder (if applicable)?
  • Was the change added to CHANGES file?

NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.

Description of change

This PR begins to resolve the many warnings output during documentation builds (recent example on Read the Docs; expand the python -m sphinx ... log line to see the warnings and errors).

There are a significant number of rendering issues caused by incorrect Napoleon formatting in docstrings, but this PR is specifically aiming to resolve some warnings and errors. Here's an example of a rendering issue caused by incorrect Napoleon formatting that needs to be addressed:

image

@gerzse gerzse merged commit 2ffcac3 into redis:master Jul 18, 2024
56 checks passed
@kurtmckee kurtmckee deleted the resolve-some-docs-warnings branch July 18, 2024 15:13
vladvildanov pushed a commit that referenced this pull request Sep 27, 2024
Get rid of some warning related to documentation.
vladvildanov pushed a commit that referenced this pull request Sep 27, 2024
Get rid of some warning related to documentation.
vladvildanov pushed a commit that referenced this pull request Sep 27, 2024
Get rid of some warning related to documentation.
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