review

Author: ldez

go.mod

@@ -131,7 +131,6 @@ require (
 	golang.org/x/exp v0.0.0-20240909161429-701f63a606c0
 	golang.org/x/mod v0.22.0
 	golang.org/x/sys v0.28.0
-	golang.org/x/text v0.20.0
 	golang.org/x/tools v0.28.0
 	gopkg.in/yaml.v3 v3.0.1
 	honnef.co/go/tools v0.5.1
@@ -200,6 +199,7 @@ require (
 	go.uber.org/zap v1.24.0 // indirect
 	golang.org/x/exp/typeparams v0.0.0-20241108190413-2d47ceb2692f // indirect
 	golang.org/x/sync v0.10.0 // indirect
+	golang.org/x/text v0.20.0 // indirect
 	google.golang.org/protobuf v1.34.2 // indirect
 	gopkg.in/ini.v1 v1.67.0 // indirect
 	gopkg.in/yaml.v2 v2.4.0 // indirect

pkg/commands/help.go

@@ -5,16 +5,15 @@ import (
 	"slices"
 	"sort"
 	"strings"
+	"unicode"
+	"unicode/utf8"
 
 	"github.com/fatih/color"
-	"github.com/spf13/cobra"
-	"golang.org/x/text/cases"
-	"golang.org/x/text/language"
-
 	"github.com/golangci/golangci-lint/pkg/config"
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
 	"github.com/golangci/golangci-lint/pkg/lint/lintersdb"
 	"github.com/golangci/golangci-lint/pkg/logutils"
+	"github.com/spf13/cobra"
 )
 
 type helpCommand struct {
@@ -126,16 +125,21 @@ func printLinters(lcs []*linter.Config) {
 	})
 
 	for _, lc := range lcs {
+		desc := lc.Linter.Desc()
+
 		// If the linter description spans multiple lines, truncate everything following the first newline
-		linterDescription := lc.Linter.Desc()
-		firstNewline := strings.IndexRune(linterDescription, '\n')
+		firstNewline := strings.IndexRune(desc, '\n')
 		if firstNewline > 0 {
-			linterDescription = linterDescription[:firstNewline]
+			desc = desc[:firstNewline]
 		}
 
-		// Capitalize the first word of the linter description
-		if firstWord, remaining, ok := strings.Cut(linterDescription, " "); ok {
-			linterDescription = cases.Title(language.Und, cases.NoLower).String(firstWord) + " " + remaining
+		rawDesc := []rune(desc)
+
+		r, _ := utf8.DecodeRuneInString(desc)
+		rawDesc[0] = unicode.ToUpper(r)
+
+		if rawDesc[len(rawDesc)-1] != '.' {
+			rawDesc = append(rawDesc, '.')
 		}
 
 		deprecatedMark := ""
@@ -144,6 +148,6 @@ func printLinters(lcs []*linter.Config) {
 		}
 
 		_, _ = fmt.Fprintf(logutils.StdOut, "%s%s: %s [fast: %t, auto-fix: %t]\n",
-			color.YellowString(lc.Name()), deprecatedMark, linterDescription, !lc.IsSlowLinter(), lc.CanAutoFix)
+			color.YellowString(lc.Name()), deprecatedMark, string(rawDesc), !lc.IsSlowLinter(), lc.CanAutoFix)
 	}
 }