`err = fmt.Errorf("Error uploading the release asset %s, %s", fi.Name(), response.Status)` and return nothing in the end could clean up the return branch a bit, since you're using named return in this method