Skip to content

Adding fixtures to series tests as per #22550 #23247

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

Closed
wants to merge 6 commits into from

Conversation

blueenvelope31
Copy link

Issue #22550

@pep8speaks
Copy link

Hello @blueenvelope31! Thanks for submitting the PR.

Line 61:80: E501 line too long (85 > 79 characters)

Line 78:80: E501 line too long (80 > 79 characters)
Line 80:80: E501 line too long (81 > 79 characters)
Line 103:80: E501 line too long (81 > 79 characters)

Line 183:80: E501 line too long (94 > 79 characters)
Line 200:80: E501 line too long (87 > 79 characters)
Line 367:80: E501 line too long (95 > 79 characters)

@WillAyd WillAyd added the Testing pandas testing functions or related to the test suite label Oct 23, 2018
@WillAyd
Copy link
Member

WillAyd commented Oct 23, 2018

Change looked OK at initial glance though not sure about test failure. Can you try merging in master?

@jreback
Copy link
Contributor

jreback commented Oct 24, 2018

can you rebase

@jreback
Copy link
Contributor

jreback commented Oct 26, 2018

can you merge master, ping on green.

@WillAyd
Copy link
Member

WillAyd commented Nov 24, 2018

Closing as stale. Ping if you'd like to continue

@WillAyd WillAyd closed this Nov 24, 2018
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.

4 participants