#60: search config file in directories from file path up to root

Author: jirfag

pkg/commands/run.go

@@ -8,12 +8,14 @@ import (
 	"io/ioutil"
 	"log"
 	"os"
+	"path/filepath"
 	"runtime"
 	"strings"
 	"time"
 
 	"github.com/fatih/color"
 	"github.com/golangci/golangci-lint/pkg/config"
+	"github.com/golangci/golangci-lint/pkg/fsutils"
 	"github.com/golangci/golangci-lint/pkg/lint"
 	"github.com/golangci/golangci-lint/pkg/lint/lintersdb"
 	"github.com/golangci/golangci-lint/pkg/printers"
@@ -318,16 +320,80 @@ func (e *Executor) parseConfig() {
 		return
 	}
 
-	if configFile == "" {
-		viper.SetConfigName(".golangci")
-		viper.AddConfigPath("./")
-	} else {
+	if configFile != "" {
 		viper.SetConfigFile(configFile)
+	} else {
+		setupConfigFileSearch(fs.Args())
 	}
 
 	e.parseConfigImpl()
 }
 
+func setupConfigFileSearch(args []string) {
+	// skip all args ([golangci-lint, run/linters]) before files/dirs list
+	for len(args) != 0 {
+		if args[0] == "run" {
+			args = args[1:]
+			break
+		}
+
+		args = args[1:]
+	}
+
+	// find first file/dir arg
+	firstArg := "./..."
+	if len(args) != 0 {
+		firstArg = args[0]
+	}
+
+	absStartPath, err := filepath.Abs(firstArg)
+	if err != nil {
+		logrus.Infof("Can't make abs path for %q: %s", firstArg, err)
+		absStartPath = filepath.Clean(firstArg)
+	}
+
+	// start from it
+	var curDir string
+	if fsutils.IsDir(absStartPath) {
+		curDir = absStartPath
+	} else {
+		curDir = filepath.Dir(absStartPath)
+	}
+
+	// find all dirs from it up to the root
+	configSearchPaths := []string{"./"}
+	for {
+		configSearchPaths = append(configSearchPaths, curDir)
+		newCurDir := filepath.Dir(curDir)
+		if curDir == newCurDir || newCurDir == "" {
+			break
+		}
+		curDir = newCurDir
+	}
+
+	logrus.Infof("Config search paths: %s", configSearchPaths)
+	viper.SetConfigName(".golangci")
+	for _, p := range configSearchPaths {
+		viper.AddConfigPath(p)
+	}
+}
+
+func getRelPath(p string) string {
+	wd, err := os.Getwd()
+	if err != nil {
+		logrus.Infof("Can't get wd: %s", err)
+		return p
+	}
+
+	r, err := filepath.Rel(wd, p)
+	if err != nil {
+		logrus.Infof("Can't make path %s relative to %s: %s", p, wd, err)
+		return p
+	}
+
+	return r
+}
+
 func (e *Executor) parseConfigImpl() {
 	commandLineConfig := *e.cfg // make copy
 
@@ -338,13 +404,24 @@ func (e *Executor) parseConfigImpl() {
 		logrus.Fatalf("Can't read viper config: %s", err)
 	}
 
+	usedConfigFile := viper.ConfigFileUsed()
+	if usedConfigFile == "" {
+		return
+	}
+	logrus.Infof("Used config file %s", getRelPath(usedConfigFile))
+
 	if err := viper.Unmarshal(&e.cfg); err != nil {
 		logrus.Fatalf("Can't unmarshal config by viper: %s", err)
 	}
 
 	if err := e.validateConfig(&commandLineConfig); err != nil {
 		logrus.Fatal(err)
 	}
+
+	if e.cfg.InternalTest { // just for testing purposes: to detect config file usage
+		fmt.Fprintln(printers.StdOut, "test")
+		os.Exit(0)
+	}
 }
 
 func (e *Executor) validateConfig(commandLineConfig *config.Config) error {

pkg/config/config.go

@@ -164,7 +164,7 @@ type Issues struct {
 	Diff              bool   `mapstructure:"new"`
 }
 
-type Config struct { // nolint:maligned
+type Config struct { //nolint:maligned
 	Run Run
 
 	Output struct {
@@ -177,4 +177,6 @@ type Config struct { // nolint:maligned
 	LintersSettings LintersSettings `mapstructure:"linters-settings"`
 	Linters         Linters
 	Issues          Issues
+
+	InternalTest bool // Option is used only for testing golangci-lint code, don't use it
 }

test/run_test.go

@@ -19,14 +19,18 @@ func installBinary(t assert.TestingT) {
 	})
 }
 
-func TestCongratsMessageIfNoIssues(t *testing.T) {
-	out, exitCode := runGolangciLint(t, "../...")
+func checkNoIssuesRun(t *testing.T, out string, exitCode int) {
 	assert.Equal(t, 0, exitCode)
 	assert.Equal(t, "Congrats! No issues were found.\n", out)
 }
 
+func TestCongratsMessageIfNoIssues(t *testing.T) {
+	out, exitCode := runGolangciLint(t, "../...")
+	checkNoIssuesRun(t, out, exitCode)
+}
+
 func TestDeadline(t *testing.T) {
-	out, exitCode := runGolangciLint(t, "--no-config", "--deadline=1ms", "../...")
+	out, exitCode := runGolangciLint(t, "--deadline=1ms", "../...")
 	assert.Equal(t, 4, exitCode)
 	assert.Equal(t, "", out) // no 'Congrats! No issues were found.'
 }
@@ -54,6 +58,19 @@ func runGolangciLint(t *testing.T, args ...string) (string, int) {
 }
 
 func TestTestsAreLintedByDefault(t *testing.T) {
-	out, exitCode := runGolangciLint(t, "--no-config", "./testdata/withtests")
+	out, exitCode := runGolangciLint(t, "./testdata/withtests")
 	assert.Equal(t, 0, exitCode, out)
 }
+
+func TestConfigFileIsDetected(t *testing.T) {
+	checkGotConfig := func(out string, exitCode int) {
+		assert.Equal(t, 0, exitCode, out)
+		assert.Equal(t, "test\n", out) // test config contains InternalTest: true, it triggers such output
+	}
+
+	checkGotConfig(runGolangciLint(t, "testdata/withconfig/pkg"))
+	checkGotConfig(runGolangciLint(t, "testdata/withconfig/..."))
+
+	out, exitCode := runGolangciLint(t) // doesn't detect when no args
+	checkNoIssuesRun(t, out, exitCode)
+}

test/testdata/withconfig/.golangci.yml

@@ -0,0 +1 @@
+InternalTest: true

test/testdata/withconfig/pkg/pkg.go

@@ -0,0 +1,3 @@
+package pkg
+
+func SomeTestFunc() {}