Skip to content

Remove _DtypeOpsMixin #26331

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
May 12, 2019
Merged

Remove _DtypeOpsMixin #26331

merged 2 commits into from
May 12, 2019

Conversation

TomAugspurger
Copy link
Contributor

No description provided.

@TomAugspurger TomAugspurger added Refactor Internal refactoring of code Clean labels May 10, 2019
@TomAugspurger TomAugspurger added this to the 0.25.0 milestone May 10, 2019
@WillAyd
Copy link
Member

WillAyd commented May 10, 2019

Seems pretty straightforward. So this is purely dead development at this point right?

@TomAugspurger
Copy link
Contributor Author

TomAugspurger commented May 10, 2019 via email

@TomAugspurger
Copy link
Contributor Author

So the bulk of acb5b49 is just moving methods from _DtypeOpsMixin to ExtensionDtype.

@jreback
Copy link
Contributor

jreback commented May 11, 2019

looks fine, merge pending CI resolution.

@jreback jreback merged commit 8747be1 into pandas-dev:master May 12, 2019
@jreback
Copy link
Contributor

jreback commented May 12, 2019

thanks @TomAugspurger

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@gfyoung gfyoung gfyoung approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
Clean Refactor Internal refactoring of code
Projects
None yet
Milestone
0.25.0
Development

Successfully merging this pull request may close these issues.
4 participants
@TomAugspurger @WillAyd @jreback @gfyoung