Simplify badgateway RoundTripper (!425) · Merge requests · GitLab.org / gitlab-workhorse · GitLab

This is an archived project. Repository and other project resources are read-only.

Jacob Vosmaer requested to merge jv-simplify-badgateway into master Sep 25, 2019

During a code review I noticed that badgateway.roundTripper.RoundTrip is unnecessarily hard to read and in my opinion, a bad code style example. It uses "naked return" with no good reason. https://github.com/golang/go/wiki/CodeReviewComments#named-result-parameters

This MR is meant to address that.