Update linter's name

Author: MirrexOne

.golangci.next.reference.yml

@@ -67,7 +67,7 @@ linters:
     - goprintffuncname
     - gosec
     - gosmopolitan
-    - gounqvet
+    - unqueryvet
     - govet
     - grouper
     - iface
@@ -180,7 +180,7 @@ linters:
     - goprintffuncname
     - gosec
     - gosmopolitan
-    - gounqvet
+    - unqueryvet
     - govet
     - grouper
     - iface
@@ -1609,7 +1609,7 @@ linters:
         - Hiragana
         - Katakana
 
-    gounqvet:
+    unqueryvet:
       # Enable SQL builder checking.
       # Default: true
       check-sql-builders: false

go.mod

@@ -148,7 +148,7 @@ require (
 	codeberg.org/chavacava/garif v0.2.0 // indirect
 	dev.gaijin.team/go/golib v0.6.0 // indirect
 	github.com/Masterminds/semver/v3 v3.3.1 // indirect
-	github.com/MirrexOne/gounqvet v1.2.0
+	github.com/MirrexOne/unqueryvet v1.2.1
 	github.com/alfatraining/structtag v1.0.0 // indirect
 	github.com/aymanbagabas/go-osc52/v2 v2.0.1 // indirect
 	github.com/beorn7/perks v1.0.1 // indirect
@@ -224,3 +224,5 @@ require (
 	gopkg.in/ini.v1 v1.67.0 // indirect
 	gopkg.in/yaml.v2 v2.4.0 // indirect
 )
+
+replace github.com/MirrexOne/unqueryvet => ../unqueryvet

go.sum

@@ -2261,7 +2261,7 @@
             }
           }
         },
-        "gounqvetSettings": {
+        "unqueryvetSettings": {
           "type": "object",
           "additionalProperties": false,
           "properties": {
@@ -4601,8 +4601,8 @@
             "gosmopolitan": {
               "$ref": "#/definitions/settings/definitions/gosmopolitanSettings"
             },
-            "gounqvet": {
-              "$ref": "#/definitions/settings/definitions/gounqvetSettings"
+            "unqueryvet": {
+              "$ref": "#/definitions/settings/definitions/unqueryvetSettings"
             },
             "govet": {
               "$ref": "#/definitions/settings/definitions/govetSettings"

jsonschema/golangci.next.jsonschema.json

@@ -86,7 +86,7 @@ var defaultLintersSettings = LintersSettings{
 		EscapeHatches:   []string{},
 		WatchForScripts: []string{"Han"},
 	},
-	Gounqvet: GounqvetSettings{
+	Unqueryvet: UnqueryvetSettings{
 		CheckSQLBuilders: true,
 	},
 	Inamedparam: INamedParamSettings{
@@ -252,7 +252,7 @@ type LintersSettings struct {
 	Gomodguard               GoModGuardSettings               `mapstructure:"gomodguard"`
 	Gosec                    GoSecSettings                    `mapstructure:"gosec"`
 	Gosmopolitan             GosmopolitanSettings             `mapstructure:"gosmopolitan"`
-	Gounqvet                 GounqvetSettings                 `mapstructure:"gounqvet"`
+	Unqueryvet                 UnqueryvetSettings                 `mapstructure:"unqueryvet"`
 	Govet                    GovetSettings                    `mapstructure:"govet"`
 	Grouper                  GrouperSettings                  `mapstructure:"grouper"`
 	Iface                    IfaceSettings                    `mapstructure:"iface"`
@@ -595,7 +595,7 @@ type GosmopolitanSettings struct {
 	WatchForScripts []string `mapstructure:"watch-for-scripts"`
 }
 
-type GounqvetSettings struct {
+type UnqueryvetSettings struct {
 	CheckSQLBuilders bool     `mapstructure:"check-sql-builders"`
 	AllowedPatterns  []string `mapstructure:"allowed-patterns"`
 }

pkg/config/linters_settings.go

@@ -1,4 +1,4 @@
-//golangcitest:args -Egounqvet
+//golangcitest:args -Eunqueryvet
 package testdata
 
 import (
@@ -7,25 +7,21 @@ import (
 	"strconv"
 )
 
-// badQueries
 func _() {
 	query := "SELECT * FROM users" // want "avoid SELECT \\* - explicitly specify needed columns for better performance, maintainability and stability"
 
 	var db *sql.DB
 	rows, _ := db.Query("SELECT * FROM orders WHERE status = ?", "active") // want "avoid SELECT \\* - explicitly specify needed columns for better performance, maintainability and stability"
 	_ = rows
 
-	// This should not trigger because it's a COUNT function
 	count := "SELECT COUNT(*) FROM users"
 	_ = count
 
-	// Good queries (should not trigger)
 	goodQuery := "SELECT id, name, email FROM users"
 	_ = goodQuery
 
 	fmt.Println(query)
 
-	// Use strconv to satisfy std lib import requirement
 	_ = strconv.Itoa(42)
 }
 
@@ -36,15 +32,12 @@ type SQLBuilder interface {
 	Query() string
 }
 
-// badSQLBuilder
 func _(builder SQLBuilder) {
 	query := builder.Select("*").From("products") // want "avoid SELECT \\* in SQL builder - explicitly specify columns to prevent unnecessary data transfer and schema change issues"
 	_ = query
 }
 
-// goodSQLBuilder
 func _(builder SQLBuilder) {
-	// Good usage - should not trigger
 	query := builder.Select("id", "name", "price").From("products")
 	_ = query
 }

pkg/golinters/gounqvet/testdata/gounqvet_custom.go

@@ -0,0 +1,29 @@
+//golangcitest:args -Eunqueryvet
+//golangcitest:config_path testdata/unqueryvet_custom.yml
+package testdata
+
+import (
+	"database/sql"
+	"fmt"
+	"strconv"
+)
+
+func _() {
+	query := "SELECT * FROM users" // want "avoid SELECT \\* - explicitly specify needed columns for better performance, maintainability and stability"
+
+	var db *sql.DB
+	rows, _ := db.Query("SELECT * FROM orders WHERE status = ?", "active") // want "avoid SELECT \\* - explicitly specify needed columns for better performance, maintainability and stability"
+	_ = rows
+
+	count := "SELECT COUNT(*) FROM users"
+	_ = count
+
+	goodQuery := "SELECT id, name, email FROM users"
+	_ = goodQuery
+
+	fmt.Println(query)
+
+	_ = strconv.Itoa(42)
+}
+
+// Custom allowed patterns test - SELECT * from temp tables should be allowed

pkg/golinters/gounqvet/testdata/gounqvet_custom.yml

@@ -0,0 +1,6 @@
+version: "2"
+
+linters:
+  settings:
+    unqueryvet:
+      check-sql-builders: false