That's what infrastructure is for.
Created attachment 43291 [details]
Created attachment 43292 [details]
Comment on attachment 43292 [details]
I think add_all should be add_patches
add() doens't need to exist.
The tests will need to update names if you do that.
There is likely some super-slick python way to do this, by implementing the right bits to be a sequence type, but that's more python than I know. I'll CC some python folks though and they can give post-landing comments if they choose.
Not quite sure what you're asking, but if it's about add vs add_all, you can do something like
def add(self, *patches):
use patches like an array
foobar.add(somepatch) # works, passes a one-length array
foobar.add(patch1, patch2) # also works
foobar.add(*patches) # can also pass arrays
so you'd have stuff like
+ def add_all_from_bug(self, bug_id):
def add_from_bug(self, bug_id):