Skip to content

DOC Use 'Axes' instead of 'axes' in axes._base.py #20942

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
Sep 2, 2021

Conversation

timhoffm
Copy link
Member

PR Summary

Towards #18726.

@timhoffm timhoffm added this to the v3.5.0 milestone Aug 29, 2021
Copy link
Member

@story645 story645 left a comment

Choose a reason for hiding this comment

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

Comments are mostly tangential to PR. Should Axis also be capitalized since it's also an object?

@timhoffm timhoffm force-pushed the doc-axes-capitalization branch from 36359ed to 7e187e9 Compare August 30, 2021 22:39
@timhoffm
Copy link
Member Author

Should Axis also be capitalized since it's also an object?

+0.1. Technically yes, but for all other cases the class name and the logical concept are very close. So it's not that important, but still a lot of work. I'm not volunteering for that :).

@timhoffm timhoffm merged commit 4e21dcf into matplotlib:master Sep 2, 2021
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Sep 2, 2021
QuLogic added a commit that referenced this pull request Sep 2, 2021
…942-on-v3.5.x

Backport PR #20942 on branch v3.5.x (DOC Use 'Axes' instead of 'axes' in axes._base.py)
tacaswell pushed a commit that referenced this pull request Oct 20, 2021
DOC Use 'Axes' instead of 'axes' in axes._base.py
ericpre pushed a commit to ericpre/matplotlib that referenced this pull request Oct 20, 2021
…tion

DOC Use 'Axes' instead of 'axes' in axes._base.py
@timhoffm timhoffm deleted the doc-axes-capitalization branch June 10, 2022 21:18
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