-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Remove pre go1.8 code and cleanup #4933
Conversation
1d73d1f
to
fe700dd
Compare
Codecov Report
@@ Coverage Diff @@
## master #4933 +/- ##
==========================================
- Coverage 62.45% 62.43% -0.02%
==========================================
Files 193 193
Lines 27641 27619 -22
==========================================
- Hits 17262 17244 -18
+ Misses 9147 9144 -3
+ Partials 1232 1231 -1
Continue to review full report at Codecov.
|
cmd/notify-webhook.go
Outdated
if resp.StatusCode != http.StatusOK && resp.StatusCode != http.StatusNoContent { | ||
return fmt.Errorf("Unable to lookup webhook endpoint %s response(%s)", urlStr, resp.Status) | ||
} | ||
return resp.Body.Close() |
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.
Shouldn't we also close resp.Body when status code != 2xx ?
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.
We could potentially do that..
fe700dd
to
1d9b1c8
Compare
We don't need certain go1.7.x custom code anymore, since we have migrated to go1.8
1d9b1c8
to
827f211
Compare
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
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
We don't need certain go1.7.x custom code anymore, since
we have migrated to go1.8
Description
Remove pre go1.8 code and cleanup
Motivation and Context
Migrate to go1.8 cleanup.
How Has This Been Tested?
Manually
Types of changes
Checklist: