Skip to content

Use timezone aware datetimes#6587

Merged
leahecole merged 11 commits intomasterfrom
tz-aware-datetimes
Nov 11, 2021
Merged

Use timezone aware datetimes#6587
leahecole merged 11 commits intomasterfrom
tz-aware-datetimes

Conversation

@busunkim96
Copy link
Copy Markdown
Contributor

@busunkim96 busunkim96 commented Aug 17, 2021

Follow up to #6581.

This was a find/replace across the repo. If a timezone aware datetime is not appropriate for your sample please let me know!

@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Aug 17, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 17, 2021
@busunkim96 busunkim96 marked this pull request as ready for review August 20, 2021 16:43
@busunkim96 busunkim96 requested review from a team, gcseh, leahecole and rachael-ds as code owners August 20, 2021 16:43
@busunkim96 busunkim96 requested a review from a team August 20, 2021 16:43
@busunkim96 busunkim96 requested a review from a team as a code owner August 20, 2021 16:43
Copy link
Copy Markdown
Collaborator

@leahecole leahecole left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for Composer

@leahecole
Copy link
Copy Markdown
Collaborator

run/idp-sql failures unrelated

@leahecole
Copy link
Copy Markdown
Collaborator

rebuilding 3.10 to see if the iot is a flake

@leahecole
Copy link
Copy Markdown
Collaborator

Confirmed 3.10 failures unrelated!

@leahecole leahecole merged commit fe3daea into master Nov 11, 2021
@leahecole leahecole deleted the tz-aware-datetimes branch November 11, 2021 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants