Skip to content

refactor: augment private getWriteStream with view support#7196

Merged
shollyman merged 1 commit intogoogleapis:mainfrom
shollyman:client-cleanup
Dec 29, 2022
Merged

refactor: augment private getWriteStream with view support#7196
shollyman merged 1 commit intogoogleapis:mainfrom
shollyman:client-cleanup

Conversation

@shollyman
Copy link
Copy Markdown
Contributor

This PR augments the base client with fuller support for view-based resolution of GetWriteStream metadata. This PR also adds an integration test that compares behaviors between different stream types (default vs explicitly created).

Towards: #7103

This PR augments the base client with fuller support for view-based
resolution of GetWriteStream metadata.  This PR also adds an integration
test that compares behaviors between different stream types (default vs
explicitly created).

Towards: googleapis#7103
@shollyman shollyman requested review from a team and mhlgio December 28, 2022 21:58
@product-auto-label product-auto-label Bot added the size: m Pull request size is medium. label Dec 28, 2022
@shollyman shollyman requested review from alvarowolfx and removed request for mhlgio December 28, 2022 21:58
@shollyman shollyman added the automerge Merge the pull request once unit tests and other checks pass. label Dec 29, 2022
@gcf-merge-on-green
Copy link
Copy Markdown
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green Bot removed the automerge Merge the pull request once unit tests and other checks pass. label Dec 29, 2022
@shollyman shollyman merged commit 92d9d4c into googleapis:main Dec 29, 2022
@shollyman shollyman deleted the client-cleanup branch December 29, 2022 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants