Skip to content

repr for PeriodIndex does not handle <=2 elements well (GH5372) #5376

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
merged 1 commit into from
Oct 31, 2013

Conversation

RomanPekar
Copy link
Contributor

closes #5372

@jreback
Copy link
Contributor

jreback commented Oct 29, 2013

pls try to name the issues and commits to something related to the issue (and just reference the issue, e.g. GH5372)

@RomanPekar
Copy link
Contributor Author

@jreback sorry, changed name to more appropriate

@jreback
Copy link
Contributor

jreback commented Oct 29, 2013

gr8

@jreback
Copy link
Contributor

jreback commented Oct 29, 2013

can you add a release notes entry refing this issue? (doc/source/release.rst)

@jreback
Copy link
Contributor

jreback commented Oct 30, 2013

pls also rebase and squash: https://github.com/pydata/pandas/wiki/Using-Git

@@ -633,6 +633,8 @@ Bug Fixes
- Fixed issue with ``drop`` and a non-unique index on Series (:issue:`5248`)
- Fixed seg fault in C parser caused by passing more names than columns in
the file. (:issue:`5156`)
- Fixed bug for ``PeriodIndex`` string representation if there're then 1 or 2
Copy link
Contributor

Choose a reason for hiding this comment

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

"there're then" ? => "there are"

@RomanPekar
Copy link
Contributor Author

@jreback hope I did it right

@jreback
Copy link
Contributor

jreback commented Oct 30, 2013

the release notes look fine, but you need to squash...

git rebase -i <commit hash before your first one>

then s or f on the commits

then

git push myfork thisbranch -f

@RomanPekar
Copy link
Contributor Author

@jreback another try, sorry, used github only occasionally before

jreback added a commit that referenced this pull request Oct 31, 2013
repr for PeriodIndex does not handle <=2 elements well (GH5372)
@jreback jreback merged commit 30dcacf into pandas-dev:master Oct 31, 2013
@jreback
Copy link
Contributor

jreback commented Oct 31, 2013

@RomanPekar thanks!

@RomanPekar RomanPekar deleted the periodIndex-repr branch October 31, 2013 03:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

repr for PeriodIndex does not handle <=2 elements well
3 participants