fix golangci failures

Signed-off-by: Benjamin Wang <wachao@vmware.com>
ahrtr · Apr 21, 2023 · c256342 · c256342
1 parent ea851ec
commit c256342
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/Makefile b/Makefile
@@ -3,10 +3,10 @@ GOFILES = $(shell find . -name \*.go)
 
 .PHONY: fmt
 fmt:
-	@echo "Verifying gofmt, failures can be fixed with ./scripts/fix.sh"
+	@echo "Verifying gofmt"
 	@!(gofmt -l -s -d ${GOFILES} | grep '[a-z]')
 
-	@echo "Verifying goimports, failures can be fixed with ./scripts/fix.sh"
+	@echo "Verifying goimports"
 	@!(go run golang.org/x/tools/cmd/goimports@latest -l -d ${GOFILES} | grep '[a-z]')
 
 .PHONY: lint

diff --git a/main.go b/main.go
@@ -96,9 +96,6 @@ func defragCommandFunc(cmd *cobra.Command, args []string) {
 		fmt.Printf("Defragmenting endpoint: %s\n", ep)
 
 		evalRet, err := evaluate(globalCfg, statusList[i])
-		if err != nil {
-
-		}
 		if !evalRet || err != nil {
 			if err != nil {
 				failures++
@@ -126,7 +123,7 @@ func defragCommandFunc(cmd *cobra.Command, args []string) {
 		ctx, cancel := commandCtx(globalCfg.commandTimeout)
 		startTs := time.Now()
 		_, err = c.Defragment(ctx, ep)
-		d := time.Now().Sub(startTs)
+		d := time.Since(startTs)
 		cancel()
 
 		if err != nil {