Merge pull request #496 from sputn1ck/dockerize_linter

Dockerize linter, add gosimports

Author: guggero

.golangci.yml

@@ -2,45 +2,134 @@ run:
   # timeout for analysis
   deadline: 4m
 
-  # Linting uses a lot of memory. Keep it under control by only running a single
-  # worker.
-  concurrency: 1
+  skip-files:
+    - "\\.pb\\.go$"
+    - "\\.pb\\.gw\\.go$"
 
 linters-settings:
+  govet:
+    # Don't report about shadowed variables
+    check-shadowing: false
+
   gofmt:
     # simplify code: gofmt with `-s` option, true by default
     simplify: true
 
+  tagliatelle:
+    case:
+      rules:
+        json: snake
+  
+  whitespace:
+    multi-func: true
+    multi-if: true
+
+  gosec:
+    excludes:
+      - G402 # Look for bad TLS connection settings.
+      - G306 # Poor file permissions used when writing to a new file.
+
+  staticcheck: 
+    go: "1.18"
+    checks: ["-SA1019"]
+
 linters:
   enable-all: true
   disable:
-  # Global variables are used in many places throughout the code base.
+   # Global variables are used in many places throughout the code base.
   - gochecknoglobals
 
   # Some lines are over 80 characters on purpose and we don't want to make them
   # even longer by marking them as 'nolint'.
   - lll
 
-  # We don't care (enough) about misaligned structs to lint that.
-  - maligned
+  # We want to allow short variable names.
+  - varnamelen
+
+  # We want to allow TODOs.
+  - godox
 
   # We have long functions, especially in tests. Moving or renaming those would
   # trigger funlen problems that we may not want to solve at that time.
   - funlen
 
   # Disable for now as we haven't yet tuned the sensitivity to our codebase
-  # yet.  Enabling by default for example, would also force new contributors to
+  # yet. Enabling by default for example, would also force new contributors to
   # potentially extensively refactor code, when they want to smaller change to
   # land.
   - gocyclo
+  - gocognit
+  - cyclop
 
   # Instances of table driven tests that don't pre-allocate shouldn't trigger
   # the linter.
   - prealloc
 
   # Init functions are used by loggers throughout the codebase.
   - gochecknoinits
+  
+  # Causes stack overflow, see https://github.com/polyfloyd/go-errorlint/issues/19.
+  - errorlint
+
+  # Deprecated linters. See https://golangci-lint.run/usage/linters/.
+  - interfacer
+  - golint
+  - maligned
+  - scopelint
+  
+  # New linters that need a code adjustment first.
+  - wrapcheck
+  - nolintlint
+  - paralleltest
+  - tparallel
+  - testpackage
+  - gofumpt
+  - gomoddirectives
+  - ireturn
+  - maintidx
+  - nlreturn
+  - dogsled
+  - gci
+  - containedctx
+  - contextcheck
+  - errname
+  - exhaustivestruct
+  - goerr113
+  - gomnd
+  - ifshort
+  - noctx
+  - nestif
+  - wsl
+  - exhaustive
+  - forcetypeassert
+  - nilerr
+  - nilnil
+  - stylecheck
+  - thelper
+
+  # Additions compared to LND
+  - exhaustruct
 
 issues:
   # Only show newly introduced problems.
   new-from-rev: 36838cf7f464cf73b0201798063b2caffeae4250
+
+  exclude-rules:
+
+      # Allow fmt.Printf() in test files
+      - path: _test\.go
+        linters:
+          - forbidigo
+        
+      # Allow fmt.Printf() in loopd
+      - path: cmd/loopd/*
+        linters:
+          - forbidigo
+      - path: loopd/*
+        linters:
+          - forbidigo
+      
+      # Allow fmt.Printf() in loop
+      - path: cmd/loop/*
+        linters:
+          - forbidigo

Makefile

@@ -1,10 +1,16 @@
 .DEFAULT_GOAL := build
 
 PKG := github.com/lightninglabs/loop
+TOOLS_DIR := tools
 
 GOTEST := GO111MODULE=on go test -v
 
+
+GOIMPORTS_PKG := github.com/rinchsan/gosimports/cmd/gosimports
+
 GO_BIN := ${GOPATH}/bin
+GOIMPORTS_BIN := $(GO_BIN)/gosimports
+
 GOBUILD := GO111MODULE=on go build -v
 GOINSTALL := GO111MODULE=on go install -v
 GOMOD := GO111MODULE=on go mod
@@ -13,51 +19,37 @@ COMMIT := $(shell git describe --abbrev=40 --dirty)
 LDFLAGS := -ldflags "-X $(PKG).Commit=$(COMMIT)"
 DEV_TAGS = dev
 
-GOFILES_NOVENDOR = $(shell find . -type f -name '*.go' -not -path "./vendor/*")
+GOFILES_NOVENDOR = $(shell find . -type f -name '*.go' -not -path "./vendor/*" -not -name "*pb.go" -not -name "*pb.gw.go" -not -name "*.pb.json.go")
 GOLIST := go list $(PKG)/... | grep -v '/vendor/'
 
-LINT_BIN := $(GO_BIN)/golangci-lint
-LINT_PKG := github.com/golangci/golangci-lint/cmd/golangci-lint
-LINT_COMMIT := v1.18.0
-LINT = $(LINT_BIN) run -v
-
-DEPGET := cd /tmp && GO111MODULE=on go get -v
 XARGS := xargs -L 1
 
 TEST_FLAGS = -test.timeout=20m
 
 UNIT := $(GOLIST) | $(XARGS) env $(GOTEST) $(TEST_FLAGS)
 
+# Linting uses a lot of memory, so keep it under control by limiting the number
+# of workers if requested.
+ifneq ($(workers),)
+LINT_WORKERS = --concurrency=$(workers)
+endif
+
+DOCKER_TOOLS = docker run -v $$(pwd):/build loop-tools
+
 GREEN := "\\033[0;32m"
 NC := "\\033[0m"
 define print
 	echo $(GREEN)$1$(NC)
 endef
 
-$(LINT_BIN):
-	@$(call print, "Fetching linter")
-	$(DEPGET) $(LINT_PKG)@$(LINT_COMMIT)
-
-unit: 
-	@$(call print, "Running unit tests.")
-	$(UNIT)
-
-fmt:
-	@$(call print, "Formatting source.")
-	gofmt -l -w -s $(GOFILES_NOVENDOR)
-
-lint: $(LINT_BIN)
-	@$(call print, "Linting source.")
-	$(LINT)
+# ============
+# DEPENDENCIES
+# ============
 
-mod-tidy:
-	@$(call print, "Tidying modules.")
-	$(GOMOD) tidy
+$(GOIMPORTS_BIN):
+	@$(call print, "Installing goimports.")
+	cd $(TOOLS_DIR); go install -trimpath $(GOIMPORTS_PKG)
 
-mod-check:
-	@$(call print, "Checking modules.")
-	$(GOMOD) tidy
-	if test -n "$$(git status | grep -e "go.mod\|go.sum")"; then echo "Running go mod tidy changes go.mod/go.sum"; git status; git diff; exit 1; fi
 
 # ============
 # INSTALLATION
@@ -94,3 +86,40 @@ clean:
 	@$(call print, "Cleaning up.")
 	rm -f ./loop-debug ./loopd-debug
 	rm -rf ./vendor
+
+# =======
+# TESTING
+# =======
+
+unit: 
+	@$(call print, "Running unit tests.")
+	$(UNIT)
+
+# =========
+# UTILITIES
+# =========
+
+fmt: $(GOIMPORTS_BIN)
+	@$(call print, "Fixing imports.")
+	gosimports -w $(GOFILES_NOVENDOR)
+	@$(call print, "Formatting source.")
+	gofmt -l -w -s $(GOFILES_NOVENDOR)
+
+lint: docker-tools
+	@$(call print, "Linting source.")
+	$(DOCKER_TOOLS) golangci-lint run -v $(LINT_WORKERS)
+
+docker-tools:
+	@$(call print, "Building tools docker image.")
+	docker build -q -t loop-tools $(TOOLS_DIR)
+
+mod-tidy:
+	@$(call print, "Tidying modules.")
+	$(GOMOD) tidy
+
+mod-check:
+	@$(call print, "Checking modules.")
+	$(GOMOD) tidy
+	if test -n "$$(git status | grep -e "go.mod\|go.sum")"; then echo "Running go mod tidy changes go.mod/go.sum"; git status; git diff; exit 1; fi
+
+

cmd/loop/liquidity.go

@@ -476,7 +476,6 @@ func setParams(ctx *cli.Context) error {
 	// so that it does not need to be manually updated.
 	case categoriesSet:
 		params.FeePpm = 0
-
 	}
 	// Update our parameters to our mutated values.
 	_, err = client.SetLiquidityParams(

cmd/loop/loopout.go

@@ -161,7 +161,7 @@ func loopOut(ctx *cli.Context) error {
 	}
 
 	// Show a warning if a slow swap was requested.
-	warning := ""
+	var warning string
 	if fast {
 		warning = "Fast swap requested."
 	} else {

cmd/loop/lsat.go

@@ -49,7 +49,6 @@ func listAuth(ctx *cli.Context) error {
 
 	tokens := make([]*printableToken, len(resp.Tokens))
 	for i, t := range resp.Tokens {
-
 		mac := &macaroon.Macaroon{}
 		err := mac.UnmarshalBinary(t.BaseMacaroon)
 		if err != nil {

cmd/loop/main.go

@@ -11,6 +11,7 @@ import (
 	"strings"
 	"time"
 
+	"github.com/btcsuite/btcd/btcutil"
 	"github.com/lightninglabs/lndclient"
 	"github.com/lightninglabs/loop"
 	"github.com/lightninglabs/loop/loopd"
@@ -21,9 +22,6 @@ import (
 	"github.com/lightninglabs/protobuf-hex-display/proto"
 	"github.com/lightningnetwork/lnd/lncfg"
 	"github.com/lightningnetwork/lnd/macaroons"
-
-	"github.com/btcsuite/btcd/btcutil"
-
 	"github.com/urfave/cli"
 	"google.golang.org/grpc"
 	"google.golang.org/grpc/credentials"

cmd/loop/swaps.go

@@ -60,7 +60,7 @@ func swapInfo(ctx *cli.Context) error {
 		id = ctx.String("id")
 	case ctx.NArg() > 0:
 		id = args[0]
-		args = args.Tail()
+		args = args.Tail() // nolint:wastedassign
 	default:
 		// Show command help if no arguments and flags were provided.
 		return cli.ShowCommandHelp(ctx, "swapinfo")

interface.go

@@ -305,7 +305,7 @@ type LoopInSwapInfo struct { // nolint
 
 // LoopOutSwapInfo contains essential information of a loop-out swap after the
 // swap is initiated.
-type LoopOutSwapInfo struct { // nolint:golint
+type LoopOutSwapInfo struct { // nolint:revive
 	// SwapHash contains the sha256 hash of the swap preimage.
 	SwapHash lntypes.Hash
 

liquidity/liquidity.go

@@ -262,7 +262,6 @@ func (p Parameters) String() string {
 		ruleList = append(
 			ruleList, fmt.Sprintf("Peer: %v: %v", peer, rule),
 		)
-
 	}
 
 	return fmt.Sprintf("rules: %v, failure backoff: %v, sweep "+

liquidity/liquidity_test.go

@@ -1409,7 +1409,6 @@ func TestSizeRestrictions(t *testing.T) {
 						Maximum: 6000,
 					}, nil,
 				).Once()
-
 			},
 			suggestions:   nil,
 			expectedError: ErrMaxExceedsServer,
@@ -1582,7 +1581,6 @@ func TestFeePercentage(t *testing.T) {
 				error) {
 
 				return testCase.quote, nil
-
 			}
 
 			lnd.Channels = []lndclient.ChannelInfo{