Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: remove warnings from reservation methods #249

Merged
merged 7 commits into from Oct 11, 2021

Conversation

dpcollins-google
Copy link
Collaborator

No description provided.

@product-auto-label product-auto-label bot added the api: pubsublite Issues related to the googleapis/python-pubsublite API. label Sep 28, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 28, 2021
@busunkim96 busunkim96 added the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 28, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 28, 2021
@tseaver
Copy link
Contributor

tseaver commented Sep 29, 2021

I would argue that Owlbot is doing more harm than good, here: the changes to the generated code shouldn't be piggybacked onto this PR. Could we tag the PR owlbot: ignore, back out its commit, and go on?

@parthea
Copy link
Contributor

parthea commented Oct 9, 2021

I reverted the changes by owlbot. I'll merge once the checks pass.

@busunkim96
Copy link
Contributor

Ignoring 3.10 samples failure as the check is still optional. All other status checks are green.

@busunkim96 busunkim96 merged commit 2defd91 into main Oct 11, 2021
@busunkim96 busunkim96 deleted the remove_reservation_warnings branch October 11, 2021 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: pubsublite Issues related to the googleapis/python-pubsublite API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants