Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Mar 31, 2026. It is now read-only.

refactor: add / use 'Client._list_resource' method#446

Merged
tseaver merged 5 commits into
mastergoogleapis/python-storage:masterfrom
38-refactor-client-_list_resourcegoogleapis/python-storage:38-refactor-client-_list_resourceCopy head branch name to clipboard
Jun 8, 2021
Merged

refactor: add / use 'Client._list_resource' method#446
tseaver merged 5 commits into
mastergoogleapis/python-storage:masterfrom
38-refactor-client-_list_resourcegoogleapis/python-storage:38-refactor-client-_list_resourceCopy head branch name to clipboard

Conversation

@tseaver

@tseaver tseaver commented May 20, 2021

Copy link
Copy Markdown
Contributor

Toward #38.

Note: this PR is based on 2acfb06 of PR #443. I will rebase to master when that PR lands. Done.

@product-auto-label product-auto-label Bot added the api: storage Issues related to the googleapis/python-storage API. label May 20, 2021
@tseaver tseaver changed the title 38 refactor client list resource refactor: add / use 'Client._list_resource' method May 20, 2021
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label May 20, 2021
@tseaver tseaver force-pushed the 38-refactor-client-_list_resource branch from 0af1e83 to 5024d1f Compare June 8, 2021 16:57
@tseaver tseaver marked this pull request as ready for review June 8, 2021 16:57
@tseaver tseaver requested review from a team June 8, 2021 16:57
@tseaver tseaver mentioned this pull request Jun 8, 2021
tseaver added 5 commits June 8, 2021 13:47
Also, adjust tests for 'Bucket.list_blobs' not to depend on anything but
calling 'Client.list_blobs'.

Toward #38
Also, provide explicit coverage for 'client._item_to_bucket' and
'client._item_to_hmack_key_metadata' helpers.

Toward #38
Also, provide explicit coverage for 'bucket._item_to_notification'.

Toward #38
@tseaver tseaver force-pushed the 38-refactor-client-_list_resource branch from 5024d1f to 68b762e Compare June 8, 2021 17:47

@chrisrossi chrisrossi 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!

@tseaver tseaver merged commit 6bd8a20 into master Jun 8, 2021
@tseaver tseaver deleted the 38-refactor-client-_list_resource branch June 8, 2021 18:25
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
* Adjust tests for 'Bucket.list_blobs' not to depend on anything but calling 'Client.list_blobs'.
* Provide explicit coverage for 'client._item_to_bucket' and 'client._item_to_hmack_key_metadata' helpers.
* Provide explicit coverage for 'bucket._item_to_notification'.

Toward googleapis#38
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
* Adjust tests for 'Bucket.list_blobs' not to depend on anything but calling 'Client.list_blobs'.
* Provide explicit coverage for 'client._item_to_bucket' and 'client._item_to_hmack_key_metadata' helpers.
* Provide explicit coverage for 'bucket._item_to_notification'.

Toward googleapis#38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: storage Issues related to the googleapis/python-storage API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.