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

Docs: get rid of "Return type: None" for procedures with no return value #277

Merged
merged 2 commits into from
Dec 21, 2021

Conversation

adamjstewart
Copy link
Collaborator

@adamjstewart adamjstewart commented Dec 13, 2021

Because we use mypy in strict mode, all methods must declare a return type. For procedures with no return value, this is None. Sphinx tries to document this by adding "Return type: None" to all procedures, but we don't really need this clutter. This PR removes that.

@adamjstewart adamjstewart added the documentation Improvements or additions to documentation label Dec 13, 2021
@adamjstewart adamjstewart added this to the 0.1.1 milestone Dec 13, 2021
@adamjstewart adamjstewart modified the milestones: 0.1.1, 0.1.2 Dec 19, 2021
@adamjstewart adamjstewart changed the title Various improvements to API docs Docs: get rid of "Return type: None" for procedures with no return value Dec 21, 2021
@adamjstewart adamjstewart marked this pull request as ready for review December 21, 2021 20:46
@adamjstewart adamjstewart mentioned this pull request Dec 21, 2021
20 tasks
@calebrob6 calebrob6 merged commit d985f31 into main Dec 21, 2021
@calebrob6 calebrob6 deleted the docs/autodoc branch December 21, 2021 22:05
@adamjstewart adamjstewart modified the milestones: 0.1.2, 0.2.0 Jan 1, 2022
@adamjstewart adamjstewart added utilities Utilities for working with geospatial data and removed utilities Utilities for working with geospatial data labels Jan 2, 2022
yichiac pushed a commit to yichiac/torchgeo that referenced this pull request Apr 29, 2023
…lue (microsoft#277)

* Various improvements to API docs

* Remove setting that is already the default
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants