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

bug(date3): rely on comparison of hour and year strings but not strict char position #138

Merged
merged 1 commit into from
Nov 2, 2015

Conversation

philiptzou
Copy link
Collaborator

Which cause certain strings with format like "MM DD, YYxx xx:SS" parsed incorrectly.

See #137

@codecov-io
Copy link

Current coverage is 73.59%

Merging #138 into master will increase coverage by +0.16% as of 193d8d6

@@            master    #138   diff @@
======================================
  Files           14      14       
  Stmts         1547    1545     -2
  Branches       292     292       
  Methods          0       0       
======================================
+ Hit           1136    1137     +1
+ Partial         93      92     -1
+ Missed         318     316     -2

Review entire Coverage Diff as of 193d8d6

Powered by Codecov. Updated on successful CI builds.

…t char position

Which cause certain strings with format like "MM DD, YYxx xx:SS" parsed
incorrectly.

See bear#137
@philiptzou philiptzou force-pushed the pull-request-date3-string-comparison branch from e528d27 to 73b4542 Compare November 2, 2015 06:00
@bear
Copy link
Owner

bear commented Nov 2, 2015

nice fix, and with tests! woo!

bear added a commit that referenced this pull request Nov 2, 2015
…parison

bug(date3): rely on comparison of hour and year strings but not strict char position
@bear bear merged commit 600df4b into bear:master Nov 2, 2015
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.

3 participants