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 into
masterfrom
64-add-client-close
Jun 14, 2021
Merged

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

Conversation

@tseaver

Copy link
Copy Markdown
Contributor

FBO use with 'contextlib.closing'.

Closes#64.

@tseaver
tseaver requested review from a team, busunkim96, chrisrossi, crwilcox and tswastJune 11, 2021 20:43
@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label Jun 11, 2021
@tseaver
tseaverforce-pushed the 64-add-client-close branch from ef3bbf0 to 090a970CompareJune 14, 2021 16:54
@tseaver
tseaverforce-pushed the 64-add-client-close branch from 51b77e2 to 71b94c6CompareJune 14, 2021 17:59
@tseavertseaver mentioned this pull request Jun 14, 2021

@chrisrossichrisrossi left a comment

Copy link
Copy Markdown

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 masterJun 14, 2021
@tseaver
tseaver deleted the 64-add-client-close branch June 14, 2021 21:12
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yesThis 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

@tseaver@chrisrossi