-
Notifications
You must be signed in to change notification settings - Fork 82
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
Update google storage library #89
Conversation
This commit updates cloud.google.com/go/storage to 1.35. Signed-off-by: Filip Petkovski <[email protected]>
Signed-off-by: Filip Petkovski <[email protected]>
5178dea
to
64d52ef
Compare
Signed-off-by: Filip Petkovski <[email protected]>
64d52ef
to
16876bc
Compare
Signed-off-by: Filip Petkovski <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
README.md
Outdated
@@ -88,8 +88,7 @@ type BucketReader interface { | |||
// IsObjNotFoundErr returns true if error means that object is not found. Relevant to Get operations. | |||
IsObjNotFoundErr(err error) bool | |||
|
|||
// IsAccessDeniedErr returns true if access to object is denied. | |||
IsAccessDeniedErr(err error) bool | |||
// IsAccessDeniedErr returns true if acces to object is denied. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems a bit off - do we need to edit the mdox line above?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, the method comment had a typo.
Signed-off-by: Filip Petkovski <[email protected]>
@GiedriusS any other comments or are we good to merge this? |
Signed-off-by: Filip Petkovski <[email protected]>
This commit updates cloud.google.com/go/storage to 1.35.
Changes
Upgrade
cloud.google.com/go/storage version
v1.35.1
Verification
Unit tests