8000 Update some mplot3d docs. by anntzer · Pull Request #14612 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]

Skip to content

Update some mplot3d docs. #14612

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

Merged
merged 1 commit into from
Jun 25, 2019
Merged

Update some mplot3d docs. #14612

merged 1 commit into from
Jun 25, 2019

Conversation

anntzer
Copy link
Contributor
@anntzer anntzer commented Jun 25, 2019

Mostly removing "This function was added, but not tested" when codecov
indicates that the function is actually covered by CI (it may still not
be directly tested, but that's the case of a bunch of APIs and we
don't bother with peppering their docstrings with such statements).

The get_zbound docstring is adapted from the one of get_xbound.

PR Summary

PR Checklist

  • Has Pytest style unit tests
  • Code is Flake 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

Mostly removing "This function was added, but not tested" when codecov
indicates that the function is actually covered by CI (it may still not
be *directly* tested, but that's the case of a bunch of APIs and we
don't bother with peppering their docstrings with such statements).

The get_zbound docstring is adapted from the one of get_xbound.
@WeatherGod WeatherGod merged commit e8b40f9 into matplotlib:master Jun 25, 2019
@anntzer anntzer deleted the 3ddocs branch June 25, 2019 14:04
@QuLogic QuLogic added this to the v3.2.0 milestone Jul 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0