Skip to content

CLN: Addtional Refactoring of MultiIndex Tests #21918

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
elmq0022 opened this issue Jul 15, 2018 · 0 comments · Fixed by #21928
Closed

CLN: Addtional Refactoring of MultiIndex Tests #21918

elmq0022 opened this issue Jul 15, 2018 · 0 comments · Fixed by #21928
Labels
Clean MultiIndex Testing pandas testing functions or related to the test suite
Milestone
0.24.0

Comments

@elmq0022
Copy link
Contributor

This is a follow up to #18644 #21514

The referenced issue and pull request split the MultiIndex tests into separate files and introduced additional fixtures. There is still outstanding comments from @jreback to finalize the cleanup.
@elmq0022 elmq0022 changed the title CLN: Addtional Refactroing of MultiIndex Tests CLN: Addtional Refactoring of MultiIndex Tests Jul 15, 2018
@jschendel jschendel added Testing pandas testing functions or related to the test suite MultiIndex Clean labels Jul 15, 2018
elmq0022 added a commit to elmq0022/pandas that referenced this issue Jul 16, 2018
@elmq0022
Addresses GH pandas-dev#21918
e2b4574
@elmq0022 elmq0022 mentioned this issue Jul 16, 2018
Merged
4 tasks
elmq0022 added a commit to elmq0022/pandas that referenced this issue Jul 17, 2018
@elmq0022
Addresses GH pandas-dev#21918
a887988 
removed test_operations.py

fixed syntax for python 2.7

fix comments from commit
@jreback jreback added this to the 0.24.0 milestone Jul 17, 2018
jreback pushed a commit that referenced this issue Jul 24, 2018
@elmq0022 @jreback
CLN: Address MulitIndex Test Follow Ups in Issue #21918 (#21928)
c9f9ee3
Sup3rGeo pushed a commit to Sup3rGeo/pandas that referenced this issue Oct 1, 2018
@elmq0022
CLN: Address MulitIndex Test Follow Ups in Issue pandas-dev#21918 (pa…
d9b48b0 
…ndas-dev#21928)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Clean MultiIndex Testing pandas testing functions or related to the test suite
Projects
None yet
Milestone
0.24.0
Development

Successfully merging a pull request may close this issue.

CLN: Address MulitIndex Test Follow Ups in Issue #21918 elmq0022/pandas
3 participants
@jreback @elmq0022 @jschendel