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

text/scanner add date scan #40182

Closed
publicocean0 opened this issue Jul 13, 2020 · 1 comment
Closed

text/scanner add date scan #40182

publicocean0 opened this issue Jul 13, 2020 · 1 comment

Comments

@publicocean0
Copy link

@publicocean0 publicocean0 commented Jul 13, 2020

1.14

It is very usefull if you can add also the possibility to scan the date format in this scanner

@ALTree
Copy link
Member

@ALTree ALTree commented Jul 13, 2020

text/scanner is an utf-8 tokenizer, it seems to me that parsing a date is an operation that is much more high level that that; you can already parse dates using time.Parse. I'm not sure a date parser belongs in text/scanner. I'm closing this issue, since I think the problem of parsing a date can already be solved using the time package, but feel free to comment if you want to explain why having one in text/scanner would be better.

@ALTree ALTree closed this Jul 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.