FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add Logging to README.md. by daspecster · Pull Request #1901 · googleapis/google-cloud-python · GitHub

Add Logging to README.md. - #1901

Merged
daspecster merged 2 commits into
googleapis:masterfrom
daspecster:add-logging-to-readme
Jun 27, 2016
Merged

Add Logging to README.md.#1901
daspecster merged 2 commits into
googleapis:masterfrom
daspecster:add-logging-to-readme

Conversation

Copy link
Copy Markdown
Contributor

@tseaver LMKWYT and if there's anything I should add.
/cc @dhermes

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 24, 2016
daspecster changed the title Add Logging to README.md. closes #1898. Add Logging to README.md. Jun 24, 2016
Comment thread README.rst
logger = client.logger('log_name')
logger.log_text("A simple entry") # API call


This comment was marked as spam.

tseaver commented Jun 26, 2016

Copy link
Copy Markdown
Contributor

LGTM, even if you decide to ignore my nitpick.

tseaver commented Jun 26, 2016

Copy link
Copy Markdown
Contributor

Hmm, maybe flesh the example out to show fetching entries? E.g.:

>>> entries, token = logger.list_entries()
>>> for entry in entries:
...   print entry.payload
A simple entry

tseaver commented Jun 26, 2016

Copy link
Copy Markdown
Contributor

@daspecster reminder to close #1898 after merge (I think having closes #1898 at the end of the first line of the commit message hides it from Github).

tseaver commented Jun 27, 2016

Copy link
Copy Markdown
Contributor

LGTM.

daspecster merged commit 5c4db0d into googleapis:master Jun 27, 2016
daspecster deleted the add-logging-to-readme branch June 27, 2016 16:18

Copy link
Copy Markdown
Contributor Author

@tseaver looks like the closes #1898 in the commit message worked!

tseaver mentioned this pull request Jun 29, 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 2, 2026
Updated `system_tests/system_tests_sync/test_default.py` and `system_tests/system_tests_async/test_default.py` to catch `RefreshError` and check for "invalid_grant" in the error message when running with `authorized_user.json` credentials. This allows the test to pass by confirming the library correctly received the error from the server, acknowledging that the CI credentials are often stale.

This builds on @daniel-sanche' work in [PR #1883](https://togithub.com/googleapis/google-auth-library-python/pull/1883).

Recent changes to `main` appear to have adjusted the response values in these tests and was leading to them failing during CI/CD presubmit tests ([see example below](https://togithub.com/googleapis/google-auth-library-python/pull/1901#issuecomment-3682277336)). This update now prevents those failing tests.


---
*PR created automatically by Jules for task [5370487295365926138](https://jules.google.com/task/5370487295365926138) started by @chalmerlowe*
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL