Skip to content

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Aug 25, 2015

@jgeewax Here is a little more auth (towards #805 and #830)

To view my and your original PRs (as we go), here are some hosted files:
www.googledrive.com/host/0B8el7dRo8mVOUVlkSEJKN3pQdkk (dhermes)
www.googledrive.com/host/0B8el7dRo8mVOOHE4NzFXSmhHUWs (jgeewax)

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 25, 2015
@dhermes
Copy link
Contributor Author

dhermes commented Aug 25, 2015

@jgeewax If you're lurking around GitHub, this PR is super tiny.

@jgeewax
Copy link
Contributor

jgeewax commented Aug 25, 2015

👍

dhermes added a commit that referenced this pull request Aug 25, 2015
Add basic discussion of clients to auth doc.
@dhermes dhermes merged commit ead33cc into googleapis:master Aug 25, 2015
@dhermes
Copy link
Contributor Author

dhermes commented Aug 25, 2015

I assume that meant LGTM. If not, I'm happy to revert.

@dhermes dhermes deleted the auth-discuss-clients branch August 25, 2015 19:42
@jgeewax
Copy link
Contributor

jgeewax commented Aug 25, 2015

Yes LGTM.

parthea pushed a commit that referenced this pull request Nov 22, 2025
* added failing test

* prevent _start_bg_task from running

* let bg task run

* invalidate transport stubs after channel refresh

* added sync implementation
parthea pushed a commit that referenced this pull request Nov 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auth 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