Skip to content
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

TST: Add a test for fillna in PeriodArray #50671

Merged
merged 6 commits into from
Jan 15, 2023

Conversation

mstazherova
Copy link
Contributor

@mstazherova mstazherova force-pushed the GH-24537/parr-fillna-test branch from e1caeb1 to 786f288 Compare January 10, 2023 23:02
@phofl phofl added the Sprints Sprint Pull Requests label Jan 10, 2023
@mstazherova mstazherova force-pushed the GH-24537/parr-fillna-test branch 2 times, most recently from b99e457 to 9f5c054 Compare January 11, 2023 08:02
@mstazherova mstazherova marked this pull request as ready for review January 11, 2023 10:11
@mstazherova
Copy link
Contributor Author

@phofl @noatamir ready for review

@mstazherova mstazherova requested a review from phofl January 12, 2023 09:04
@mstazherova mstazherova force-pushed the GH-24537/parr-fillna-test branch from ad941d7 to 6085795 Compare January 12, 2023 09:20
@mstazherova mstazherova force-pushed the GH-24537/parr-fillna-test branch from 6085795 to 5f494f0 Compare January 14, 2023 01:41
@mstazherova
Copy link
Contributor Author

@phofl should I still do something before this PR can be merged? Right now it's failing on pre-commit, but it's not related to my changes.

@simonjayhawkins simonjayhawkins changed the title ENH: Add a test for fillna in PeriodArray TST: Add a test for fillna in PeriodArray Jan 14, 2023
@phofl
Copy link
Member

phofl commented Jan 14, 2023

Sorry, was a bit busy yesterday.

pre-commit is failing, otherwise this lgtm

@phofl
Copy link
Member

phofl commented Jan 15, 2023

Looks like pre-commit failure came from something else, merged main to check

@phofl phofl added this to the 2.0 milestone Jan 15, 2023
@phofl phofl merged commit 9e8a243 into pandas-dev:main Jan 15, 2023
@phofl
Copy link
Member

phofl commented Jan 15, 2023

thx @mstazherova

@mstazherova
Copy link
Contributor Author

Cool, thank you! Excited to do more :)

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

Successfully merging this pull request may close these issues.

Float conversion makes fillna lossy
2 participants