marcacohen/boto — comment on commit 62eb29d3d510b943a31ec2084d9ac45910d92fe3
File: boto/gs/bucket.py line 56
By mfschwartz:

I'm guessing you have version_id in here because this helper is also called from get_acl(), which has that named param defined on the parent class. (That's not necessary, given Python's "duck typing"). In general I don't like having params in a provider-specific subclass that don't make sense in that subclass. Or if there's a compelling reason to have them in the interface, at least verify they weren't specfied in the provider-specific subclass where they don't make sense.