-
Notifications
You must be signed in to change notification settings - Fork 1k
printtz #3500
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
printtz #3500
Conversation
…g' 2) Added more tests 3) fixed some minor spacing issues
|
In line with @franknarf1 's comment in #1523 : |
|
ISO 8601 timestamps would be ideal IMO:
Many other languages (e.g. Spark, Presto, Go) also understand the It would be amazing if |
|
Thanks @fparages! I've invited you to be project member. (You'll need to accept the invite before GitHub adds you.) |
Codecov Report
@@ Coverage Diff @@
## master #3500 +/- ##
==========================================
+ Coverage 96.66% 96.66% +<.01%
==========================================
Files 65 65
Lines 12278 12288 +10
==========================================
+ Hits 11868 11878 +10
Misses 410 410
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #3500 +/- ##
==========================================
+ Coverage 96.66% 96.66% +<.01%
==========================================
Files 65 65
Lines 12278 12288 +10
==========================================
+ Hits 11868 11878 +10
Misses 410 410
Continue to review full report at Codecov.
|
Allows printing of timezones when printing data.tables with POSIX columns (if tzone attribute is defined).
Closes #2842. An item in #1523 list.
Defined as optional via a parameter timezone with default value = FALSE, for backwards compatibility.