Skip to content

REF: use _get_axis_number instead of if axis in [...] #43220

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

REF: use _get_axis_number instead of if axis in [...] #43220

wants to merge 1 commit into from

Conversation

goyaldhara
Copy link

@goyaldhara goyaldhara commented Aug 25, 2021 •
edited
Loading

  • Added _get_axis_number in style.py as per the standards

@pep8speaks
Copy link

Hello @goyaldhara! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 1168:89: E501 line too long (127 > 88 characters)
Line 1168:113: E261 at least two spaces before inline comment

@goyaldhara goyaldhara mentioned this pull request Aug 25, 2021
Closed
3 tasks
@MarcoGorelli
Copy link
Member

looks like an exact copy of #43121 with a trivial edit to make it look like it's your own

@goyaldhara goyaldhara deleted the REF-43048 branch August 26, 2021 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@goyaldhara @pep8speaks @MarcoGorelli