-
Notifications
You must be signed in to change notification settings - Fork 13
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
When converting a date time where the time is '00:00:00' infer returns a date type instead of date time #11
Conversation
Remove if dt.time() to avoid convenrting timestampsat midnight to dates
…and added 3.5 check
…times Do not use dupers on py > 3.5 and add extra timezone conversions
@bruth could you have a look into my pull request please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor formatting, but looks great otherwise. Thanks for the contribution! I will look into the coverage issue.
@@ -1,11 +1,15 @@ | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove empty line.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
strconv.py
Outdated
from datetime import datetime | ||
import re | ||
|
||
import sys |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Move import of re
and sys
to the top with spaces.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you mean with "with spaces"?
Solved this by removing if if dt.time() in the date_formats loop in the convert_datetime() method.