Skip to content

CLN: put mgr_locs setter next to property definition #19264

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 1 commit into from
Jan 17, 2018
Merged

CLN: put mgr_locs setter next to property definition #19264

merged 1 commit into from
Jan 17, 2018

Conversation

jbrockmendel
Copy link
Member

  • closes #xxxx
  • tests added / passed
  • passes git diff upstream/master -u -- "*.py" | flake8 --diff
  • whatsnew entry

@jreback jreback added Internals Related to non-user accessible pandas implementation Clean labels Jan 16, 2018
@jreback jreback added this to the 0.23.0 milestone Jan 16, 2018
@jreback
Copy link
Contributor

jreback commented Jan 16, 2018

lgtm.

@TomAugspurger
Copy link
Contributor

I don't know why this bugged me so much, but it did :)

@jorisvandenbossche jorisvandenbossche changed the title put mgr_locs setter next to property definition CLN: put mgr_locs setter next to property definition Jan 16, 2018
@jreback jreback merged commit 04a6815 into pandas-dev:master Jan 17, 2018
@jreback
Copy link
Contributor

jreback commented Jan 17, 2018

thanks!

@jbrockmendel jbrockmendel deleted the internals_cleanup2 branch February 11, 2018 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Clean Internals Related to non-user accessible pandas implementation
Projects
None yet
Milestone
0.23.0
Development

Successfully merging this pull request may close these issues.
3 participants
@jbrockmendel @jreback @TomAugspurger