fread tz= default changed from "" to "UTC" #4894
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Follow up to #4464
Yes, you were right, @MichaelChirico.
At least I did say at the bottom of this comment that I wasn't as strongly against tz='UTC' by default and that we could ask the community.
News item contains the detail.
The next release containing this change would be 1.14.0.
Merging now straight away without review. We can see if it passes GLCI, I'll start a rerun of all revdeps just to make sure, and I'll tweet asking community to test dev and raise any objections. We can keep discussing and see if any objections are raised. We can always revert this PR if necessary. Merging is necessary so that folk can test master.
Update:
GLCI passed and deployed, so folk can test master now.
Community consulted via tweet: https://twitter.com/MattDowle/status/1358011599336931328
Revdep rerun complete and looks good; i.e. no impact. These 8 are either error/warning on CRAN already for unrelated reasons, or we've communicated with maintainer before and it's a known different issue.