Skip to content

ENH Make year and quarter dash-separatable #9701

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
Mar 26, 2015

Conversation

williamhogman
Copy link
Contributor

This closes #9688 by making years and quarters separable by dashes.

@jreback jreback added Enhancement Period Period data type labels Mar 23, 2015
@@ -3018,6 +3018,21 @@ def test_parse_time_string():
assert reso == reso_lower


def test_parse_time_quarter_w_dash():
pairs = [
Copy link
Contributor

Choose a reason for hiding this comment

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

add the issue in a comment here

Copy link
Contributor

Choose a reason for hiding this comment

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

are there possibilities that it shouldn't match?

@williamhogman
Copy link
Contributor Author

@jreback I think I've adjusted them accordingly, although I'm not quite sure about the how you want the test-case class. Anyhow, tell me if it looks OK so I can rebase it :)

assert parsed == parsed_lower
assert reso == reso_lower
class TestParseTimeString(tm.TestCase):

Copy link
Contributor

Choose a reason for hiding this comment

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

add the issue number as a comment here

@jreback jreback added this to the 0.16.1 milestone Mar 25, 2015
@jreback
Copy link
Contributor

jreback commented Mar 25, 2015

minor comments, pls squash as well. thanks. ping when green.

This closes pandas-dev#9688 by making years and quarters separable by dashes.
@williamhogman williamhogman force-pushed the feature-dashed-quarters branch from c620a51 to 91c63a6 Compare March 26, 2015 05:55
@williamhogman
Copy link
Contributor Author

@jreback That should do it, commits are now squashed.

jreback added a commit that referenced this pull request Mar 26, 2015
ENH Make year and quarter dash-separatable
@jreback jreback merged commit 8d2818e into pandas-dev:master Mar 26, 2015
@jreback
Copy link
Contributor

jreback commented Mar 26, 2015

thank you!

@williamhogman
Copy link
Contributor Author

Happy to help, pandas has saved my ass too many times to count.

William Rudenmalm
tel. +46 70 263 86 33
www.whn.se

On Thu, Mar 26, 2015 at 10:50 AM, jreback [email protected] wrote:

thank you!


Reply to this email directly or view it on GitHub
#9701 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Period Period data type
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ENH: make period_range parse strings like '2012-Q1', not just '2012Q1'
2 participants