feat(cmd): remove global name scheme, prevent using lints

Author: juliusmh

.golangci.yml

@@ -41,6 +41,7 @@ linters:
     - unused
     - usestdlibvars
     - whitespace
+    - forbidigo
 
   exclusions:
     paths:
@@ -169,6 +170,19 @@ linters:
         - float-compare
         - go-require
       enable-all: true
+    forbidigo:
+      analyze-types: true
+      forbid:
+        - pattern: ^.*model\.NameValidationScheme$
+          pkg: ^github.com/prometheus/common/model$
+          msg: it is deprecated. Use model/validation.NamingScheme instead.
+        - pattern: ^.*model\.LabelName\.IsValid$
+          pkg: ^github.com/prometheus/common/model$
+          msg: it depends on model.NameValidationScheme. Use model/validation.NamingScheme instead.
+        - pattern: ^.*model\.IsValidMetricName$
+          pkg: ^github.com/prometheus/common/model$
+          msg: it depends on model.NameValidationScheme. Use model/validation.NamingScheme instead.
+
 
 output:
   show-stats: false

cmd/prometheus/main.go

@@ -141,10 +141,6 @@ var (
 )
 
 func init() {
-	// This can be removed when the legacy global mode is fully deprecated.
-	//nolint:staticcheck
-	model.NameValidationScheme = model.UTF8Validation
-
 	prometheus.MustRegister(versioncollector.NewCollector(strings.ReplaceAll(appName, "-", "_")))
 
 	var err error

cmd/prometheus/main_test.go

@@ -46,12 +46,6 @@ import (
 	"github.com/prometheus/prometheus/util/testutil"
 )
 
-func init() {
-	// This can be removed when the legacy global mode is fully deprecated.
-	//nolint:staticcheck
-	model.NameValidationScheme = model.UTF8Validation
-}
-
 const startupTime = 10 * time.Second
 
 var (

cmd/promtool/main.go

@@ -63,12 +63,6 @@ import (
 
 var promqlEnableDelayedNameRemoval = false
 
-func init() {
-	// This can be removed when the legacy global mode is fully deprecated.
-	//nolint:staticcheck
-	model.NameValidationScheme = model.UTF8Validation
-}
-
 const (
 	successExitCode = 0
 	failureExitCode = 1

cmd/promtool/main_test.go

@@ -39,12 +39,6 @@ import (
 	"github.com/prometheus/prometheus/promql/promqltest"
 )
 
-func init() {
-	// This can be removed when the legacy global mode is fully deprecated.
-	//nolint:staticcheck
-	model.NameValidationScheme = model.UTF8Validation
-}
-
 var promtoolPath = os.Args[0]
 
 func TestMain(m *testing.M) {