Group imports by package.

internal/errorutil/errors.go

@@ -1,6 +1,8 @@
 package errorutil
 
-import "fmt"
+import (
+	"fmt"
+)
 
 // PanicError can be used to not print stacktrace twice
 type PanicError struct {

internal/pkgcache/pkgcache.go

@@ -9,14 +9,12 @@ import (
 	"sort"
 	"sync"
 
-	"github.com/golangci/golangci-lint/pkg/logutils"
-
-	"github.com/golangci/golangci-lint/pkg/timeutils"
-
 	"github.com/pkg/errors"
 	"golang.org/x/tools/go/packages"
 
 	"github.com/golangci/golangci-lint/internal/cache"
+	"github.com/golangci/golangci-lint/pkg/logutils"
+	"github.com/golangci/golangci-lint/pkg/timeutils"
 )
 
 // Cache is a per-package data cache. A cached data is invalidated when

pkg/commands/config.go

@@ -4,12 +4,11 @@ import (
 	"fmt"
 	"os"
 
+	"github.com/spf13/cobra"
 	"github.com/spf13/viper"
 
 	"github.com/golangci/golangci-lint/pkg/exitcodes"
 	"github.com/golangci/golangci-lint/pkg/fsutils"
-
-	"github.com/spf13/cobra"
 )
 
 func (e *Executor) initConfig() {

pkg/commands/executor.go

@@ -5,19 +5,16 @@ import (
 	"github.com/spf13/cobra"
 	"github.com/spf13/pflag"
 
-	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis/load"
-
 	"github.com/golangci/golangci-lint/internal/pkgcache"
-	"github.com/golangci/golangci-lint/pkg/timeutils"
-
-	"github.com/golangci/golangci-lint/pkg/fsutils"
-
 	"github.com/golangci/golangci-lint/pkg/config"
+	"github.com/golangci/golangci-lint/pkg/fsutils"
+	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis/load"
 	"github.com/golangci/golangci-lint/pkg/goutil"
 	"github.com/golangci/golangci-lint/pkg/lint"
 	"github.com/golangci/golangci-lint/pkg/lint/lintersdb"
 	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/report"
+	"github.com/golangci/golangci-lint/pkg/timeutils"
 )
 
 type Executor struct {

pkg/commands/run.go

@@ -10,9 +10,6 @@ import (
 	"strings"
 	"time"
 
-	"github.com/golangci/golangci-lint/pkg/packages"
-	"github.com/golangci/golangci-lint/pkg/result/processors"
-
 	"github.com/fatih/color"
 	"github.com/pkg/errors"
 	"github.com/spf13/cobra"
@@ -23,8 +20,10 @@ import (
 	"github.com/golangci/golangci-lint/pkg/lint"
 	"github.com/golangci/golangci-lint/pkg/lint/lintersdb"
 	"github.com/golangci/golangci-lint/pkg/logutils"
+	"github.com/golangci/golangci-lint/pkg/packages"
 	"github.com/golangci/golangci-lint/pkg/printers"
 	"github.com/golangci/golangci-lint/pkg/result"
+	"github.com/golangci/golangci-lint/pkg/result/processors"
 )
 
 func getDefaultIssueExcludeHelp() string {

pkg/config/reader.go

@@ -7,12 +7,11 @@ import (
 	"path/filepath"
 	"strings"
 
+	homedir "github.com/mitchellh/go-homedir"
 	"github.com/spf13/viper"
 
 	"github.com/golangci/golangci-lint/pkg/fsutils"
 	"github.com/golangci/golangci-lint/pkg/logutils"
-
-	homedir "github.com/mitchellh/go-homedir"
 )
 
 type FileReader struct {

pkg/fsutils/filecache.go

@@ -5,9 +5,9 @@ import (
 	"io/ioutil"
 	"sync"
 
-	"github.com/golangci/golangci-lint/pkg/logutils"
-
 	"github.com/pkg/errors"
+
+	"github.com/golangci/golangci-lint/pkg/logutils"
 )
 
 type FileCache struct {

pkg/golinters/depguard.go

@@ -5,33 +5,31 @@ import (
 	"strings"
 	"sync"
 
+	"github.com/OpenPeeDeeP/depguard"
 	"golang.org/x/tools/go/analysis"
 	"golang.org/x/tools/go/loader"
 
 	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
-
-	depguardAPI "github.com/OpenPeeDeeP/depguard"
-
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 	"github.com/golangci/golangci-lint/pkg/result"
 )
 
-func setDepguardListType(dg *depguardAPI.Depguard, lintCtx *linter.Context) error {
+func setDepguardListType(dg *depguard.Depguard, lintCtx *linter.Context) error {
 	listType := lintCtx.Settings().Depguard.ListType
 	var found bool
-	dg.ListType, found = depguardAPI.StringToListType[strings.ToLower(listType)]
+	dg.ListType, found = depguard.StringToListType[strings.ToLower(listType)]
 	if !found {
 		if listType != "" {
 			return fmt.Errorf("unsure what list type %s is", listType)
 		}
-		dg.ListType = depguardAPI.LTBlacklist
+		dg.ListType = depguard.LTBlacklist
 	}
 
 	return nil
 }
 
-func setupDepguardPackages(dg *depguardAPI.Depguard, lintCtx *linter.Context) {
+func setupDepguardPackages(dg *depguard.Depguard, lintCtx *linter.Context) {
-	if dg.ListType == depguardAPI.LTBlacklist {
+	if dg.ListType == depguard.LTBlacklist {
 		// if the list type was a blacklist the packages with error messages should
 		// be included in the blacklist package list
 
@@ -66,7 +64,7 @@ func NewDepguard() *goanalysis.Linter {
 		dgSettings := &lintCtx.Settings().Depguard
 		analyzer.Run = func(pass *analysis.Pass) (interface{}, error) {
 			prog := goanalysis.MakeFakeLoaderProgram(pass)
-			dg := &depguardAPI.Depguard{
+			dg := &depguard.Depguard{
 				Packages:      dgSettings.Packages,
 				IncludeGoRoot: dgSettings.IncludeGoRoot,
 			}
@@ -87,7 +85,7 @@ func NewDepguard() *goanalysis.Linter {
 				return nil, nil
 			}
 			msgSuffix := "is in the blacklist"
-			if dg.ListType == depguardAPI.LTWhitelist {
+			if dg.ListType == depguard.LTWhitelist {
 				msgSuffix = "is not in the whitelist"
 			}
 			res := make([]result.Issue, 0, len(issues))

pkg/golinters/errcheck.go

@@ -10,15 +10,13 @@ import (
 	"strings"
 	"sync"
 
-	"golang.org/x/tools/go/analysis"
+	errcheck "github.com/golangci/errcheck/golangci"
-
-	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
-
-	errcheckAPI "github.com/golangci/errcheck/golangci"
 	"github.com/pkg/errors"
+	"golang.org/x/tools/go/analysis"
 
 	"github.com/golangci/golangci-lint/pkg/config"
 	"github.com/golangci/golangci-lint/pkg/fsutils"
+	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 	"github.com/golangci/golangci-lint/pkg/result"
 )
@@ -44,7 +42,7 @@ func NewErrcheck() *goanalysis.Linter {
 			if err != nil {
 				return nil, err
 			}
-			errcheckIssues, err := errcheckAPI.RunWithConfig(prog, errCfg)
+			errcheckIssues, err := errcheck.RunWithConfig(prog, errCfg)
 			if err != nil {
 				return nil, err
 			}
@@ -106,13 +104,13 @@ func parseIgnoreConfig(s string) (map[string]*regexp.Regexp, error) {
 	return cfg, nil
 }
 
-func genConfig(errCfg *config.ErrcheckSettings) (*errcheckAPI.Config, error) {
+func genConfig(errCfg *config.ErrcheckSettings) (*errcheck.Config, error) {
 	ignoreConfig, err := parseIgnoreConfig(errCfg.Ignore)
 	if err != nil {
 		return nil, errors.Wrap(err, "failed to parse 'ignore' directive")
 	}
 
-	c := &errcheckAPI.Config{
+	c := &errcheck.Config{
 		Ignore:  ignoreConfig,
 		Blank:   errCfg.CheckAssignToBlank,
 		Asserts: errCfg.CheckTypeAssertions,

pkg/golinters/funlen.go

@@ -5,13 +5,12 @@ import (
 	"strings"
 	"sync"
 
+	"github.com/ultraware/funlen"
 	"golang.org/x/tools/go/analysis"
 
 	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 	"github.com/golangci/golangci-lint/pkg/result"
-
-	"github.com/ultraware/funlen"
 )
 
 const funlenLinterName = "funlen"

pkg/golinters/goanalysis/runner.go

@@ -28,20 +28,16 @@ import (
 	"sync/atomic"
 	"time"
 
-	"golang.org/x/tools/go/types/objectpath"
+	"github.com/pkg/errors"
-
+	"golang.org/x/tools/go/analysis"
 	"golang.org/x/tools/go/gcexportdata"
+	"golang.org/x/tools/go/packages"
+	"golang.org/x/tools/go/types/objectpath"
 
 	"github.com/golangci/golangci-lint/internal/errorutil"
 	"github.com/golangci/golangci-lint/internal/pkgcache"
-
 	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis/load"
 	"github.com/golangci/golangci-lint/pkg/logutils"
-
-	"github.com/pkg/errors"
-
-	"golang.org/x/tools/go/analysis"
-	"golang.org/x/tools/go/packages"
 )
 
 var (

pkg/golinters/gochecknoglobals.go

@@ -10,7 +10,6 @@ import (
 	"golang.org/x/tools/go/analysis"
 
 	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
-
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 	"github.com/golangci/golangci-lint/pkg/result"
 )

pkg/golinters/gocritic.go

@@ -10,14 +10,11 @@ import (
 	"strings"
 	"sync"
 
+	"github.com/go-lintpack/lintpack"
 	"golang.org/x/tools/go/analysis"
 
-	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
-
 	"github.com/golangci/golangci-lint/pkg/config"
-
+	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
-	"github.com/go-lintpack/lintpack"
-
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 	"github.com/golangci/golangci-lint/pkg/result"
 )

pkg/golinters/godox.go

@@ -6,7 +6,6 @@ import (
 	"sync"
 
 	"github.com/matoous/godox"
-
 	"golang.org/x/tools/go/analysis"
 
 	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"

pkg/golinters/gosec.go

@@ -8,14 +8,12 @@ import (
 	"strconv"
 	"sync"
 
+	"github.com/securego/gosec"
+	"github.com/securego/gosec/rules"
 	"golang.org/x/tools/go/analysis"
 	"golang.org/x/tools/go/packages"
 
 	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
-
-	"github.com/securego/gosec"
-	"github.com/securego/gosec/rules"
-
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 	"github.com/golangci/golangci-lint/pkg/result"
 )

pkg/golinters/govet.go

@@ -3,10 +3,6 @@ package golinters
 import (
 	"golang.org/x/tools/go/analysis"
 
-	"github.com/golangci/golangci-lint/pkg/config"
-
-	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
-
 	// analysis plug-ins
 	"golang.org/x/tools/go/analysis/passes/asmdecl"
 	"golang.org/x/tools/go/analysis/passes/assign"
@@ -40,6 +36,9 @@ import (
 	"golang.org/x/tools/go/analysis/passes/unreachable"
 	"golang.org/x/tools/go/analysis/passes/unsafeptr"
 	"golang.org/x/tools/go/analysis/passes/unusedresult"
+
+	"github.com/golangci/golangci-lint/pkg/config"
+	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
 )
 
 func getAllAnalyzers() []*analysis.Analyzer {

pkg/golinters/govet_test.go

@@ -4,8 +4,6 @@ import (
 	"sort"
 	"testing"
 
-	"github.com/golangci/golangci-lint/pkg/config"
-
 	"golang.org/x/tools/go/analysis"
 	"golang.org/x/tools/go/analysis/passes/asmdecl"
 	"golang.org/x/tools/go/analysis/passes/assign"
@@ -14,6 +12,8 @@ import (
 	"golang.org/x/tools/go/analysis/passes/buildtag"
 	"golang.org/x/tools/go/analysis/passes/cgocall"
 	"golang.org/x/tools/go/analysis/passes/shadow"
+
+	"github.com/golangci/golangci-lint/pkg/config"
 )
 
 func TestGovet(t *testing.T) {

pkg/golinters/ineffassign.go

@@ -4,12 +4,10 @@ import (
 	"fmt"
 	"sync"
 
+	"github.com/golangci/ineffassign"
 	"golang.org/x/tools/go/analysis"
 
 	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
-
-	ineffassignAPI "github.com/golangci/ineffassign"
-
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 	"github.com/golangci/golangci-lint/pkg/result"
 )
@@ -37,7 +35,7 @@ func NewIneffassign() *goanalysis.Linter {
 				fileNames = append(fileNames, pos.Filename)
 			}
 
-			issues := ineffassignAPI.Run(fileNames)
+			issues := ineffassign.Run(fileNames)
 			if len(issues) == 0 {
 				return nil, nil
 			}

pkg/golinters/interfacer.go

@@ -5,11 +5,9 @@ import (
 
 	"golang.org/x/tools/go/analysis"
 	"golang.org/x/tools/go/analysis/passes/buildssa"
-
-	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
-
 	"mvdan.cc/interfacer/check"
 
+	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 	"github.com/golangci/golangci-lint/pkg/result"
 )

pkg/golinters/megacheck.go

@@ -3,9 +3,9 @@ package golinters
 import (
 	"fmt"
 
-	"github.com/golangci/golangci-lint/pkg/logutils"
-
 	"golang.org/x/tools/go/analysis"
+
+	"github.com/golangci/golangci-lint/pkg/logutils"
 )
 
 var debugf = logutils.Debug("megacheck")

pkg/golinters/unused.go

@@ -4,10 +4,9 @@ import (
 	"golang.org/x/tools/go/analysis"
 	"honnef.co/go/tools/unused"
 
+	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 	"github.com/golangci/golangci-lint/pkg/result"
-
-	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
 )
 
 func NewUnused() *goanalysis.Linter {

pkg/golinters/whitespace.go

@@ -5,13 +5,12 @@ import (
 	"sync"
 
 	"github.com/pkg/errors"
+	"github.com/ultraware/whitespace"
 	"golang.org/x/tools/go/analysis"
 
 	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 	"github.com/golangci/golangci-lint/pkg/result"
-
-	"github.com/ultraware/whitespace"
 )
 
 func NewWhitespace() *goanalysis.Linter {

pkg/lint/lintersdb/enabled_set.go

@@ -3,9 +3,8 @@ package lintersdb
 import (
 	"sort"
 
-	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
-
 	"github.com/golangci/golangci-lint/pkg/config"
+	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 	"github.com/golangci/golangci-lint/pkg/logutils"
 )

pkg/lint/lintersdb/manager.go

@@ -4,7 +4,6 @@ import (
 	"os"
 
 	"github.com/golangci/golangci-lint/pkg/config"
-
 	"github.com/golangci/golangci-lint/pkg/golinters"
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 )

pkg/lint/load.go

@@ -11,17 +11,14 @@ import (
 	"strings"
 	"time"
 
-	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis/load"
-
-	"github.com/golangci/golangci-lint/internal/pkgcache"
-
-	"github.com/golangci/golangci-lint/pkg/fsutils"
-
 	"github.com/pkg/errors"
 	"golang.org/x/tools/go/packages"
 
+	"github.com/golangci/golangci-lint/internal/pkgcache"
 	"github.com/golangci/golangci-lint/pkg/config"
 	"github.com/golangci/golangci-lint/pkg/exitcodes"
+	"github.com/golangci/golangci-lint/pkg/fsutils"
+	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis/load"
 	"github.com/golangci/golangci-lint/pkg/goutil"
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 	"github.com/golangci/golangci-lint/pkg/logutils"

pkg/lint/runner.go

@@ -7,13 +7,11 @@ import (
 	"strings"
 
 	"github.com/golangci/golangci-lint/internal/errorutil"
-	"github.com/golangci/golangci-lint/pkg/lint/lintersdb"
-
-	"github.com/golangci/golangci-lint/pkg/fsutils"
-
 	"github.com/golangci/golangci-lint/pkg/config"
+	"github.com/golangci/golangci-lint/pkg/fsutils"
 	"github.com/golangci/golangci-lint/pkg/goutil"
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
+	"github.com/golangci/golangci-lint/pkg/lint/lintersdb"
 	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/packages"
 	"github.com/golangci/golangci-lint/pkg/result"

pkg/result/issue.go

@@ -1,6 +1,8 @@
 package result
 
-import "go/token"
+import (
+	"go/token"
+)
 
 type Range struct {
 	From, To int

pkg/result/processors/exclude_rules.go

@@ -3,10 +3,8 @@ package processors
 import (
 	"regexp"
 
-	"github.com/golangci/golangci-lint/pkg/logutils"
-
 	"github.com/golangci/golangci-lint/pkg/fsutils"
-
+	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/result"
 )
 

pkg/result/processors/exclude_rules_test.go

@@ -5,10 +5,9 @@ import (
 	"path/filepath"
 	"testing"
 
-	"github.com/golangci/golangci-lint/pkg/fsutils"
-
 	"github.com/stretchr/testify/assert"
 
+	"github.com/golangci/golangci-lint/pkg/fsutils"
 	"github.com/golangci/golangci-lint/pkg/result"
 )
 

pkg/result/processors/fixer.go

@@ -8,16 +8,13 @@ import (
 	"sort"
 	"strings"
 
-	"github.com/golangci/golangci-lint/pkg/timeutils"
-
-	"github.com/golangci/golangci-lint/pkg/fsutils"
-
-	"github.com/golangci/golangci-lint/pkg/logutils"
-
 	"github.com/pkg/errors"
 
 	"github.com/golangci/golangci-lint/pkg/config"
+	"github.com/golangci/golangci-lint/pkg/fsutils"
+	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/result"
+	"github.com/golangci/golangci-lint/pkg/timeutils"
 )
 
 type Fixer struct {

pkg/result/processors/max_same_issues.go

@@ -4,7 +4,6 @@ import (
 	"sort"
 
 	"github.com/golangci/golangci-lint/pkg/config"
-
 	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/result"
 )

pkg/result/processors/processor.go

@@ -1,6 +1,8 @@
 package processors
 
-import "github.com/golangci/golangci-lint/pkg/result"
+import (
+	"github.com/golangci/golangci-lint/pkg/result"
+)
 
 type Processor interface {
 	Process(issues []result.Issue) ([]result.Issue, error)

pkg/result/processors/uniq_by_line_test.go

@@ -5,7 +5,6 @@ import (
 	"testing"
 
 	"github.com/golangci/golangci-lint/pkg/config"
-
 	"github.com/golangci/golangci-lint/pkg/result"
 )
 

test/bench/bench_test.go

@@ -13,12 +13,11 @@ import (
 	"testing"
 	"time"
 
-	"github.com/golangci/golangci-lint/test/testshared"
-
 	gops "github.com/mitchellh/go-ps"
 	"github.com/shirou/gopsutil/process"
 
 	"github.com/golangci/golangci-lint/pkg/config"
+	"github.com/golangci/golangci-lint/test/testshared"
 )
 
 func chdir(b *testing.B, dir string) {

test/data.go

@@ -1,6 +1,8 @@
 package test
 
-import "path/filepath"
+import (
+	"path/filepath"
+)
 
 const testdataDir = "testdata"
 const binName = "../golangci-lint"

test/linters_test.go

@@ -9,11 +9,10 @@ import (
 	"strings"
 	"testing"
 
-	"github.com/golangci/golangci-lint/test/testshared"
-
 	assert "github.com/stretchr/testify/require"
-
 	yaml "gopkg.in/yaml.v2"
+
+	"github.com/golangci/golangci-lint/test/testshared"
 )
 
 func runGoErrchk(c *exec.Cmd, files []string, t *testing.T) {

test/run_test.go

@@ -6,12 +6,10 @@ import (
 	"testing"
 
 	"github.com/stretchr/testify/assert"
-
+	_ "github.com/valyala/quicktemplate"
-	"github.com/golangci/golangci-lint/test/testshared"
 
 	"github.com/golangci/golangci-lint/pkg/exitcodes"
-
+	"github.com/golangci/golangci-lint/test/testshared"
-	_ "github.com/valyala/quicktemplate"
 )
 
 func getCommonRunArgs() []string {