Skip to content
This repository was archived by the owner on Feb 13, 2026. It is now read-only.

feat: add 'Client.close'#100

Merged
tseaver merged 3 commits intomasterfrom
64-add-client-close
Jun 14, 2021
Merged

feat: add 'Client.close'#100
tseaver merged 3 commits intomasterfrom
64-add-client-close

Conversation

@tseaver
Copy link
Copy Markdown
Contributor

@tseaver tseaver commented Jun 11, 2021

FBO use with 'contextlib.closing'.

Closes #64.

@tseaver tseaver requested a review from a team as a code owner June 11, 2021 20:43
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jun 11, 2021
@tseaver tseaver force-pushed the 64-add-client-close branch from ef3bbf0 to 090a970 Compare June 14, 2021 16:54
@tseaver tseaver force-pushed the 64-add-client-close branch from 51b77e2 to 71b94c6 Compare June 14, 2021 17:59
@tseaver tseaver mentioned this pull request Jun 14, 2021
Copy link
Copy Markdown

@chrisrossi chrisrossi left a comment

Choose a reason for hiding this comment

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

LGTM! I hadn't heard of contextlib.closing.

@tseaver tseaver merged commit 7387fc8 into master Jun 14, 2021
@tseaver tseaver deleted the 64-add-client-close branch June 14, 2021 21:12
gcf-merge-on-green bot pushed a commit that referenced this pull request Jun 14, 2021
🤖 I have created a release \*beep\* \*boop\*
---
## [1.7.0](https://www.github.com/googleapis/python-cloud-core/compare/v1.6.0...v1.7.0) (2021-06-14)


### Features

* add 'Client.close' ([#100](https://www.github.com/googleapis/python-cloud-core/issues/100)) ([7387fc8](https://www.github.com/googleapis/python-cloud-core/commit/7387fc87186defff74612cf85909c7d456155d47))
* add 'Client.from_service_account_info' factory ([#54](https://www.github.com/googleapis/python-cloud-core/issues/54)) ([7e59360](https://www.github.com/googleapis/python-cloud-core/commit/7e59360a43418947cbbb7608aa23ef5d566c964c))


### Documentation

* update python contributing guide ([#80](https://www.github.com/googleapis/python-cloud-core/issues/80)) ([58deef8](https://www.github.com/googleapis/python-cloud-core/commit/58deef8ce2e90415a4c38022e88fdfe25bc666b6))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clients do not clean up Requests sessions

2 participants