TST: update incorrectly defined fixtures #34971
Conversation
| Hello @pizzathief! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:
|
| df = DataFrame({'floats': N / 10 * Series(np.random.random(N)), | ||
| 'letters': Series(random_letters)}) | ||
| return df | ||
| return df_letters_var() |
There was a problem hiding this comment.
you can leave this like it was and feed df_letters into a new fixture
| | ||
| @pytest.fixture | ||
| def epoch_1960_as_pydatetime(): | ||
| # for origin as 1960-01-01 |
There was a problem hiding this comment.
can you give a doc-string to these (change the comment to a quoted string)
| df = s_main_dtypes_var() | ||
| return df | ||
| | ||
| def s_main_dtypes_items(): |
There was a problem hiding this comment.
you can inject the fixture to subsequent fixtures
jreback changed the title 3 participants
handles the deprecationwarning messages and allows "pytest pandas" to proceed. (see issue #34970)