novemberkilo opened a new pull request #955: URL: https://github.com/apache/arrow-rs/pull/955
# Which issue does this PR close? Closes #901 # What changes are included in this PR? Replaces the function `using_chrono_tz` with a function `using_chrono_tz_and_utc_naive_datetime` The rationale for this is that an offset for a timezone depends on the point in time at which the offset is required -- it will vary depending on whether DST is in effect at that point in time or not. # Are there any user-facing changes? I'm not sure. `using_chrono_tz` is a publicly available function (albeit under the `chrono-tz` feature flag) so strictly speaking the API is changing? If this is a concern, we could have both functions appear (with `using_chrono_tz` remaining unused). What do you think @alamb ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
