Skip to content

Update the helper _datetime_to_rfc3339 to handle time zones. - #2041

Merged
dhermes merged 2 commits into
googleapis:masterfrom
supriyagarg:rfc3339
Aug 1, 2016
Merged

Update the helper _datetime_to_rfc3339 to handle time zones.#2041
dhermes merged 2 commits into
googleapis:masterfrom
supriyagarg:rfc3339

Conversation

@supriyagarg

Copy link
Copy Markdown
Contributor

The helper function _format_timestamp in gcloud.monitoring.query is
superseded by this updated helper function.

The helper function _format_timestamp in gcloud.monitoring.query is
superseded by this updated helper function.
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 1, 2016
Comment threadgcloud/_helpers.py Outdated
:rtype: str
:returns: The string representing the datetime stamp.
"""
if not(ignore_zone or value.tzinfo is None):

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@dhermes

Copy link
Copy Markdown
Contributor

Mostly LGTM other than the nit about the not

Thanks a lot for doing this!

@dhermes

Copy link
Copy Markdown
Contributor

LGTM once Travis passes

@dhermes
dhermes merged commit 5a6ba14 into googleapis:masterAug 1, 2016
@supriyagarg
supriyagarg deleted the rfc3339 branch August 1, 2016 22:53
This was referenced Aug 3, 2016
parthea added a commit that referenced this pull request Nov 24, 2025
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
parthea pushed a commit that referenced this pull request Mar 6, 2026
* Migrate to pyproject.toml
* Update
* Add copyright notice
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Update pyproject.toml
---------
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Chalmer Lowe <chalmerlowe@google.com>
parthea pushed a commit that referenced this pull request Apr 1, 2026
* test: Add unit test for get_python_version
* fix
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@supriyagarg@dhermes@googlebot