Skip to content

TST: Fix error in assert_produces_warning. #3927

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

Conversation

jtratner
Copy link
Contributor

When I copied this over and sent to @cpcloud,
I thought the indentation was wrong and changed it.
Turned out I was wrong -- sorry about that!

Now all the doctests pass...

When I copied this over and sent to @cpcloud,
I thought the indentation was wrong and changed it.
Turned out I was wrong -- sorry about that!

Now all the doctests pass...
@cpcloud
Copy link
Member

cpcloud commented Jun 16, 2013

@jtratner which doc tests are u referring to?

@jtratner
Copy link
Contributor Author

the doctests in assert_produces_warning. run python -m doctest -v pandas/util/testing.py and you'll see what I mean.

On Sun, Jun 16, 2013 at 7:33 PM, Phillip Cloud notifications@github.comwrote:

@jtratner https://github.com/jtratner which doc tests are u referring
to?


Reply to this email directly or view it on GitHubhttps://github.com//pull/3927#issuecomment-19521197
.

@cpcloud
Copy link
Member

cpcloud commented Jun 16, 2013

k this is good 2 go...any objections? @jreback ?

@jreback
Copy link
Contributor

jreback commented Jun 16, 2013

ok

@cpcloud
Copy link
Member

cpcloud commented Jun 17, 2013

cool so @jtratner rebase and i will merge

@jtratner
Copy link
Contributor Author

@cpcloud I think this is actually up to date.

cpcloud added a commit that referenced this pull request Jun 17, 2013
…t-produces-warning

TST: Fix error in assert_produces_warning.
@cpcloud cpcloud merged commit 70fe0a2 into pandas-dev:master Jun 17, 2013
@cpcloud
Copy link
Member

cpcloud commented Jun 17, 2013

thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants