-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
Use empty_frame fixture #33161
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
Labels
Comments
Hi, I'm new to open source. Can I take this issue? |
Sure!
On Mar 30, 2020, at 8:10 PM, Jason-M-Chan ***@***.***> wrote:
Hi, I'm new to open source. Can I take this issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#33161 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAEU4ULUQBVIAP4X6VEGNHTRKFNJHANCNFSM4LXE4V3Q>.
William Ayd
[email protected]
|
This was referenced Mar 31, 2020
can I take a look at this issue? |
anish29292
pushed a commit
to anish29292/pandas
that referenced
this issue
Mar 31, 2020
Replaced Empty data frames with fixture empty_frame()
Fixture has been removed, closing. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
#33148 introduced an empty_frame fixture in the top level conftest. At the same time, we have a lot of
... = DataFrame()
assignments in our tests. We should standardize to use the fixture where appropriateThe text was updated successfully, but these errors were encountered: