Skip to content

Conversation

@pep8speaks
Copy link

pep8speaks commented Jun 19, 2022

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

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2022-06-21 13:48:48 UTC

@jackgoldsmith4
Copy link
Contributor Author

@github-actions pre-commit

Copy link
Member

@mroeschke mroeschke left a comment

Choose a reason for hiding this comment

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

LGTM feel free to merge when ready @phofl

@mroeschke mroeschke added the Testing pandas testing functions or related to the test suite label Jun 21, 2022
@mroeschke mroeschke added this to the 1.5 milestone Jun 21, 2022
@phofl phofl merged commit c5a640d into pandas-dev:main Jun 21, 2022
@phofl
Copy link
Member

phofl commented Jun 21, 2022

thx @jackgoldsmith4

@jackgoldsmith4 jackgoldsmith4 deleted the add-tests branch June 21, 2022 23:50
yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
* multiindex lexsort depth test

* add periodindex loc test

* import

* pep

* Fixes from pre-commit [automated commit]

* fix assert

* fewer elements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Testing pandas testing functions or related to the test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Intersection of MultiIndexes reports incorrect lexsort_depth BUG: loc behaves incorrectly on PeriodIndex within MultiIndex with 3 levels

4 participants