Skip to content

Commit

Permalink
chore: Remove the S1038 rule of the gosimple linter. (#129)
Browse files Browse the repository at this point in the history
- S1038 : Unnecessarily complex way of printing formatted string. Which said instead of using fmt.Print(fmt.Sprintf(...)), one can use fmt.Printf(...).
  • Loading branch information
shahzadlone authored Jan 24, 2022
1 parent 99bdbdd commit fdacde2
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
5 changes: 4 additions & 1 deletion .golangci.sourceinc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -440,7 +440,10 @@ linters-settings:
# Select the Go version to target. The default is '1.13'.
go: "1.15"
# https://staticcheck.io/docs/options#checks
checks: [ "all" ]
checks: [ "all", "-S1038" ]
# Turn on all except (these are disabled):
# - S1038 - Unnecessarily complex way of printing formatted string. Which said
# instead of using fmt.Print(fmt.Sprintf(...)), one can use fmt.Printf(...).

govet:
# report about shadowed variables
Expand Down
1 change: 0 additions & 1 deletion db/tests/utils.go
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,6 @@ func ExecuteQueryTestCase(t *testing.T, schema string, collectionNames []string,

for _, dbi := range dbs {
fmt.Println("--------------")
//nolint:gosimple
fmt.Println(fmt.Sprintf("Running tests with database type: %s", dbi.name))

db := dbi.db
Expand Down

0 comments on commit fdacde2

Please sign in to comment.