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.

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

Bug in date parsing? #10817

Closed
jiahao opened this issue Apr 14, 2015 · 3 comments
Closed

Bug in date parsing? #10817

jiahao opened this issue Apr 14, 2015 · 3 comments
Labels
dates Dates, times, and the Dates stdlib module

Comments

@jiahao
Copy link
Member

jiahao commented Apr 14, 2015

This looks like a bug:

julia> Date("Apr 01 2014", "uuu dd yyyy")
2014-04-01

julia> Date("Apr 01 xx 2014", "uuu dd zz yyyy")
0001-04-01

julia> Date("Apr 01 xx 2014", "uuu dd    yyyy")
0001-04-01
@jiahao jiahao added the bug Indicates an unexpected problem or unintended behavior label May 8, 2015
@jiahao
Copy link
Member Author

jiahao commented May 8, 2015

cc @quinnj

@vtjnash
Copy link
Member

vtjnash commented Jul 20, 2016

seems to be fixed now (throws "ArgumentError: Non-digit character encountered")

@vtjnash vtjnash added needs tests Unit tests are required for this change and removed bug Indicates an unexpected problem or unintended behavior labels Jul 20, 2016
@tkelman tkelman added the dates Dates, times, and the Dates stdlib module label Jul 20, 2016
@wookay wookay mentioned this issue Aug 5, 2016
Merged
tkelman added a commit that referenced this issue Aug 6, 2016
@tkelman
Merge pull request #17836 from wookay/test_date_parsing
7a66efb 
Add tests for date parsing (#10817)
ranjanan pushed a commit to ranjanan/julia that referenced this issue Aug 8, 2016
@wookay @ranjanan
Add tests for date parsing (JuliaLang#10817)
7e3f52d
@tkelman tkelman removed the needs tests Unit tests are required for this change label Aug 11, 2016
@tkelman
Copy link
Contributor

tkelman commented Aug 11, 2016

tests added by #17836

@tkelman tkelman closed this as completed Aug 11, 2016
tkelman pushed a commit that referenced this issue Aug 11, 2016
@wookay @tkelman
Add tests for date parsing (#10817)
0bbe027 
(cherry picked from commit f2863ef)
ref #17836
mfasi pushed a commit to mfasi/julia that referenced this issue Sep 5, 2016
@wookay @mfasi
Add tests for date parsing (JuliaLang#10817)
cb3c2bb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dates Dates, times, and the Dates stdlib module
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jiahao @vtjnash @tkelman