Skip to content

Commit

Permalink
importas: add message if settings contain no aliases (#1956)
Browse files Browse the repository at this point in the history
  • Loading branch information
StephenBrown2 committed May 6, 2021
1 parent d252670 commit 07a0568
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 0 deletions.
3 changes: 3 additions & 0 deletions pkg/golinters/importas.go
Expand Up @@ -24,6 +24,9 @@ func NewImportAs(settings *config.ImportAsSettings) *goanalysis.Linter {
if settings == nil {
return
}
if len(settings.Alias) == 0 {
lintCtx.Log.Infof("importas settings found, but no aliases listed. List aliases under alias: key.") // nolint: misspell
}

err := analyzer.Flags.Set("no-unaliased", strconv.FormatBool(settings.NoUnaliased))
if err != nil {
Expand Down
4 changes: 4 additions & 0 deletions test/testdata/configs/importas_noalias.yml
@@ -0,0 +1,4 @@
linters-settings:
importas:
fff: fmt
std_os: os
15 changes: 15 additions & 0 deletions test/testdata/importas_noalias.go
@@ -0,0 +1,15 @@
//args: -Eimportas
//config_path: testdata/configs/importas_noalias.yml
package testdata

import (
wrong_alias "fmt"
"os"
wrong_alias_again "os"
)

func ImportAsNoAlias() {
wrong_alias.Println("foo")
wrong_alias_again.Stdout.WriteString("bar")
os.Stdout.WriteString("test")
}

0 comments on commit 07a0568

Please sign in to comment.