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

STYLE: fix pylint style checks for inherit-non-class. #48897

Merged
merged 5 commits into from
Oct 2, 2022

Conversation

kostyafarber
Copy link
Contributor

@kostyafarber kostyafarber commented Sep 30, 2022

Added an ignore statement in the base.py. Looks like we need to inherit a non-class in this instance.

pandas/core/arrays/base.py Outdated Show resolved Hide resolved
@MarcoGorelli MarcoGorelli added the Code Style Code style, linting, code_checks label Oct 1, 2022
@MarcoGorelli MarcoGorelli added this to the 1.6 milestone Oct 1, 2022
Copy link
Member

@MarcoGorelli MarcoGorelli left a comment

Choose a reason for hiding this comment

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

looks good to me pending green

@kostyafarber
Copy link
Contributor Author

Thanks for reviewing quickly 😊

@MarcoGorelli MarcoGorelli merged commit b0237ed into pandas-dev:main Oct 2, 2022
@kostyafarber kostyafarber deleted the issue-48855 branch October 2, 2022 16:00
jbrockmendel pushed a commit to jbrockmendel/pandas that referenced this pull request Oct 3, 2022
* fix pylint for non-class-inherit and add pre-commit changes

* move ExtensionArraySupportsAnyAll to below ExtensionArray

* move if TYPE_CHECKING back up to line 94

* remove erroneous space
@kostyafarber
Copy link
Contributor Author

Hi, @MarcoGorelli this might sound like a silly request, but is it possible to add the hacktoberfest label to this PR?
Many thanks 😄

mroeschke pushed a commit that referenced this pull request Oct 11, 2022
)

* BUG: Timedelta.__new__

* remove assertion

* GH refs

* API: Timedelta(td64_obj) retain resolution

* API: Timedelta constructor pytimedelta, Tick preserve reso

* STYLE: fix pylint style checks for inherit-non-class. (#48897)

* fix pylint for non-class-inherit and add pre-commit changes

* move ExtensionArraySupportsAnyAll to below ExtensionArray

* move if TYPE_CHECKING back up to line 94

* remove erroneous space

* BUG: preserve DTA/TDA+timedeltalike scalar with mismatched resos

* merge main

* remove debugging variable

Co-authored-by: Kostya Farber <73378227+kostyafarber@users.noreply.github.com>
@mroeschke mroeschke modified the milestones: 1.6, 2.0 Oct 13, 2022
noatamir pushed a commit to noatamir/pandas that referenced this pull request Nov 9, 2022
* fix pylint for non-class-inherit and add pre-commit changes

* move ExtensionArraySupportsAnyAll to below ExtensionArray

* move if TYPE_CHECKING back up to line 94

* remove erroneous space
noatamir pushed a commit to noatamir/pandas that referenced this pull request Nov 9, 2022
…das-dev#48923)

* BUG: Timedelta.__new__

* remove assertion

* GH refs

* API: Timedelta(td64_obj) retain resolution

* API: Timedelta constructor pytimedelta, Tick preserve reso

* STYLE: fix pylint style checks for inherit-non-class. (pandas-dev#48897)

* fix pylint for non-class-inherit and add pre-commit changes

* move ExtensionArraySupportsAnyAll to below ExtensionArray

* move if TYPE_CHECKING back up to line 94

* remove erroneous space

* BUG: preserve DTA/TDA+timedeltalike scalar with mismatched resos

* merge main

* remove debugging variable

Co-authored-by: Kostya Farber <73378227+kostyafarber@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Style Code style, linting, code_checks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants