Thanks to visit codestin.com
Credit goes to github.com

Skip to content

return if the batch is empty #119

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

Merged
merged 1 commit into from
Jul 30, 2015

Conversation

zeapo
Copy link

@zeapo zeapo commented Jul 30, 2015

The batch execute when not given an http object will look into the given requests. If no request was given, it will raise a ValueError("Missing a valid http object.").

@googlebot
Copy link
Collaborator

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@googlebot googlebot added the cla: no This human has *not* signed the Contributor License Agreement. label Jul 30, 2015
@zeapo
Copy link
Author

zeapo commented Jul 30, 2015

I signed it!

@googlebot
Copy link
Collaborator

CLAs look good, thanks!

@googlebot googlebot added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels Jul 30, 2015
@nathanielmanistaatgoogle
Copy link
Contributor

This looks alright, but what use case is this supporting? How did this situation arise for you?

@zeapo
Copy link
Author

zeapo commented Jul 30, 2015

We loop through a list of users and batch add them to a group. The users to add have to respect a certain criteria, and we expected that if none of them was to be added, to have nothing happen.

@nathanielmanistaatgoogle
Copy link
Contributor

Okay. Thanks for the contribution!

nathanielmanistaatgoogle added a commit that referenced this pull request Jul 30, 2015
Return if the batch is empty in BatchHttpRequest.execute.
@nathanielmanistaatgoogle nathanielmanistaatgoogle merged commit df43dbf into googleapis:master Jul 30, 2015
akrherz pushed a commit to akrherz/google-api-python-client that referenced this pull request Apr 1, 2019
Update type on run_flow() to be accurate
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants