Bump golangci-lint version to 1.16.0 (thrasher-corp#311)

cranktakular · May 27, 2019 · e8bc3b5 · e8bc3b5
1 parent 993b73f
commit e8bc3b5
Show file tree

Hide file tree

Showing 8 changed files with 10 additions and 14 deletions.
diff --git a/.appveyor.yml b/.appveyor.yml
@@ -28,7 +28,7 @@ before_test:
 
 test_script:
   # test back-end
-  - go get github.com/golangci/golangci-lint/cmd/golangci-lint@v1.15.0
+  - go get github.com/golangci/golangci-lint/cmd/golangci-lint@v1.16.0
   - '%GOPATH%\bin\golangci-lint.exe run --verbose'
   - go test -race ./...
 

diff --git a/.golangci.yml b/.golangci.yml
@@ -78,4 +78,4 @@ issues:
         - gosec
 
 service:
-  golangci-lint-version: 1.15.x 
+  golangci-lint-version: 1.16.x 
diff --git a/Makefile b/Makefile
@@ -1,6 +1,6 @@
 LDFLAGS = -ldflags "-w -s"
 GCTPKG = github.com/thrasher-/gocryptotrader
-LINTPKG = github.com/golangci/golangci-lint/cmd/golangci-lint@v1.15.0
+LINTPKG = github.com/golangci/golangci-lint/cmd/golangci-lint@v1.16.0
 LINTBIN = $(GOPATH)/bin/golangci-lint
 GCTLISTENPORT=9050
 GCTPROFILERLISTENPORT=8085

diff --git a/config/config_test.go b/config/config_test.go
@@ -848,7 +848,7 @@ func TestReadConfig(t *testing.T) {
 
 	err = readConfig.ReadConfig("bla")
 	if err == nil {
-		t.Error("Test failed. TestReadConfig " + err.Error())
+		t.Error("Test failed. TestReadConfig error cannot be nil")
 	}
 
 	err = readConfig.ReadConfig("")

diff --git a/exchanges/btcmarkets/btcmarkets_wrapper.go b/exchanges/btcmarkets/btcmarkets_wrapper.go
@@ -232,6 +232,7 @@ func (b *BTCMarkets) CancelAllOrders(_ *exchange.OrderCancellation) (exchange.Ca
 		orderIDInt, err := strconv.ParseInt(openOrders[i].ID, 10, 64)
 		if err != nil {
 			cancelAllOrdersResponse.OrderStatus[openOrders[i].ID] = err.Error()
+			continue
 		}
 		orderList = append(orderList, orderIDInt)
 	}
@@ -243,8 +244,8 @@ func (b *BTCMarkets) CancelAllOrders(_ *exchange.OrderCancellation) (exchange.Ca
 		}
 
 		for i := range orders {
-			if err != nil {
-				cancelAllOrdersResponse.OrderStatus[strconv.FormatInt(orders[i].ID, 10)] = err.Error()
+			if !orders[i].Success {
+				cancelAllOrdersResponse.OrderStatus[strconv.FormatInt(orders[i].ID, 10)] = orders[i].ErrorMessage
 			}
 		}
 	}

diff --git a/exchanges/btse/btse_websocket.go b/exchanges/btse/btse_websocket.go
@@ -47,10 +47,6 @@ func (b *BTSE) WsConnect() error {
 			b.Name, err)
 	}
 
-	if err != nil {
-		return err
-	}
-
 	go b.WsHandleData()
 	b.GenerateDefaultSubscriptions()
 

diff --git a/exchanges/huobi/huobi_wrapper.go b/exchanges/huobi/huobi_wrapper.go
@@ -47,7 +47,7 @@ func (h *HUOBI) Run() {
 		if common.StringDataContains(h.BaseCurrencies.Strings(), "CNY") {
 			cfg := config.GetConfig()
 			exchCfg, errCNY := cfg.GetExchangeConfig(h.Name)
-			if err != nil {
+			if errCNY != nil {
 				log.Errorf("%s failed to get exchange config. %s\n", h.Name, errCNY)
 				return
 			}

diff --git a/exchanges/localbitcoins/localbitcoins_wrapper.go b/exchanges/localbitcoins/localbitcoins_wrapper.go
@@ -192,9 +192,8 @@ func (l *LocalBitcoins) SubmitOrder(p currency.Pair, side exchange.OrderSide, _
 	if err != nil {
 		return submitOrderResponse, err
 	}
-	if err == nil {
-		submitOrderResponse.IsOrderPlaced = true
-	}
+
+	submitOrderResponse.IsOrderPlaced = true
 
 	// Now to figure out what ad we just submitted
 	// The only details we have are the params above