Skip to content

ENH: GH-35611 Tests for top-level Pandas functions serializable #35692

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

Merged

Conversation

ikedaosushi
Copy link
Contributor

@ikedaosushi ikedaosushi commented Aug 12, 2020

@@ -157,3 +158,10 @@ def test_version_tag():
raise ValueError(
"No git tags exist, please sync tags between upstream and your repo"
)


def test_serializable():
Copy link
Contributor

Choose a reason for hiding this comment

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

can you parameterize the test on these instead or inside

also pls add the issue number here as a comment

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I understand. I'll do that.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jreback Could you take a look of this?

@jreback jreback added IO Pickle read_pickle, to_pickle Testing pandas testing functions or related to the test suite labels Aug 13, 2020
@ikedaosushi ikedaosushi force-pushed the ENH_GH35611_top-level-serializable branch from 8bd86ed to f8215c4 Compare August 18, 2020 14:23
@jreback jreback modified the milestones: 1.1.2, 1.2 Aug 19, 2020
@jreback jreback merged commit 71dfed7 into pandas-dev:master Aug 19, 2020
@jreback
Copy link
Contributor

jreback commented Aug 19, 2020

thanks @ikedaosushi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IO Pickle read_pickle, to_pickle Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ENH: Make top-level Pandas functions serializable
2 participants