Skip to content

Conversation

@mroeschke mroeschke changed the title Tests/old issues TST: Old Issues May 26, 2021
@mroeschke mroeschke added the Testing pandas testing functions or related to the test suite label May 26, 2021
@simonjayhawkins simonjayhawkins added this to the 1.3 milestone May 26, 2021

def test_bool_frame_mult_float():
# GH 18549
result = DataFrame(True, list("ab"), list("cd")) * 1.0
Copy link
Member

Choose a reason for hiding this comment

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

nit, I think the result assignment should be just the op under test

Suggested change
result = DataFrame(True, list("ab"), list("cd")) * 1.0
df = DataFrame(True, list("ab"), list("cd"))
result = df * 1.0

@simonjayhawkins simonjayhawkins removed this from the 1.3 milestone May 26, 2021
@jreback jreback added this to the 1.3 milestone May 26, 2021
@jreback jreback merged commit fd38824 into pandas-dev:master May 26, 2021
@jreback
Copy link
Contributor

jreback commented May 26, 2021

thanks @mroeschke

@mroeschke mroeschke deleted the tests/old_issues branch May 26, 2021 18:33
TLouf pushed a commit to TLouf/pandas that referenced this pull request Jun 1, 2021
JulianWgs pushed a commit to JulianWgs/pandas that referenced this pull request Jul 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment