Fix azure.NewFutureFromResponse to always return tracker (#334)

* Fix azure.NewFutureFromResponse to always return tracker

When creating a future always include the polling tracker even if
there's a failure; this allows the underlying response to be obtained
by the caller.

* Fix linter

.travis.yml

@@ -20,7 +20,7 @@ before_install:
   - curl -L -o $GOPATH/bin/dep https://github.com/golang/dep/releases/download/v$DEP_VERSION/dep-linux-amd64 && chmod +x $GOPATH/bin/dep
 
 install:
-  - go get -u github.com/golang/lint/golint
+  - go get -u golang.org/x/lint/golint
   - go get -u github.com/stretchr/testify
   - go get -u github.com/GoASTScanner/gas
   - dep ensure

CHANGELOG.md

@@ -1,5 +1,11 @@
 # CHANGELOG
 
+## v11.1.1
+
+### Bug Fixes
+
+- When creating a future always include the polling tracker even if there's a failure; this allows the underlying response to be obtained by the caller.
+
 ## v11.1.0
 
 ### New Features

autorest/azure/async.go

@@ -58,10 +58,7 @@ func NewFuture(req *http.Request) Future {
 // with the initial response from an asynchronous operation.
 func NewFutureFromResponse(resp *http.Response) (Future, error) {
 	pt, err := createPollingTracker(resp)
-	if err != nil {
-		return Future{}, err
-	}
-	return Future{pt: pt}, nil
+	return Future{pt: pt}, err
 }
 
 // Response returns the last HTTP response.

version/version.go

@@ -20,7 +20,7 @@ import (
 )
 
 // Number contains the semantic version of this SDK.
-const Number = "v11.1.0"
+const Number = "v11.1.1"
 
 var (
 	userAgent = fmt.Sprintf("Go/%s (%s-%s) go-autorest/%s",