bji / libs3

Other
154 stars 151 forks source link

added S3 versioning support #70

Open alexeip0 opened 6 years ago

alexeip0 commented 6 years ago
bji commented 6 years ago

Thank you for your great changes, as you can see i don't really have anything to do with this project any more and I have not been particularly good about merging pull requests in a timely fashion, but your work is appreciated. Please just address the minor issue I raised above and I will merge this request.

alexeip0 commented 6 years ago

Sorry, having hard time finding the minor issue I have to address. Is there a comment somewhere I'm not seeing?

bji commented 6 years ago

The minor issue is the question I asked above ... you have documented the version id as being 'null' if the object was created before versioning was available ... I think that the comment should be updated because it's ambiguous: 'null' could mean the string "null" or it could mean the zero-value null ... just wanted the docs to be clearer there.