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

review: DateTimeProvider - fix date times #21

Closed
sckott opened this issue Mar 9, 2017 · 0 comments
Closed

review: DateTimeProvider - fix date times #21

sckott opened this issue Mar 9, 2017 · 0 comments
Labels
Milestone

Comments

@sckott
Copy link
Collaborator

sckott commented Mar 9, 2017

This class powers ch_timezone(), ch_unix_time(), and ch_date_time(). It is not documented that the date-times are integers converted to POSIX times, so they are all after 1970.

@sckott sckott added the review label Mar 9, 2017
@sckott sckott closed this as completed in afadc7d May 25, 2017
@sckott sckott modified the milestone: v0.1 May 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant