Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Typo fixes in pyplot.py #14978

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
Aug 6, 2019
Merged

Typo fixes in pyplot.py #14978

merged 1 commit into from
Aug 6, 2019

Conversation

joaonsg
Copy link
Contributor

@joaonsg joaonsg commented Aug 4, 2019

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

@ImportanceOfBeingErnest
Copy link
Member

May I suggest to not put in a new PR for each individual typo, but maybe collect a few of them as you read along the docs and then file one PR for all of them?

Of course we are thankful for just any improvement of the documentation; on the other hand, merging a PR also creates some overhead. I would hence leave this open for now and you can add further small changes. Then ping us once you're ready.

@joaonsg
Copy link
Contributor Author

joaonsg commented Aug 4, 2019

Yes, I will do that.

@tacaswell tacaswell added this to the v3.2.0 milestone Aug 5, 2019
@timhoffm timhoffm changed the title Update pyplot.py Typo fixes in pyplot.py Aug 5, 2019
Copy link
Member

@dstansby dstansby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@dstansby dstansby merged commit bcdcb93 into matplotlib:master Aug 6, 2019
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.

4 participants