fix: gochecknoinits using init as name

pet-sitter · Aug 19, 2024 · 4feca45 · 4feca45
1 parent 17a51ee
commit 4feca45
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/.golangci.yml b/.golangci.yml
@@ -617,3 +617,7 @@ issues:
         - gosec
         - noctx
         - wrapcheck
+    - path: internal/tests/config.go # https://github.com/golangci/golangci-lint/issues/4697
+      text: "don't use `init` function"
+    - path: internal/configs/server.go
+      text: "don't use `init` function"
diff --git a/internal/configs/server.go b/internal/configs/server.go
@@ -65,7 +65,7 @@ var (
 	BreedsGoogleSheetsID = os.Getenv("BREEDS_GOOGLE_SHEETS_ID")
 )
 
-//nolint:gochecknoinits
+//nolint:init // initializes required environment variables
 func init() {
 	if Port == "" {
 		Port = "8080"

diff --git a/internal/tests/config.go b/internal/tests/config.go
@@ -4,7 +4,7 @@ import "os"
 
 var TestDatabaseURL = os.Getenv("TEST_DATABASE_URL")
 
-//nolint:gochecknoinits
+//nolint:init // initializes required environment variables
 func init() {
 	if TestDatabaseURL == "" {
 		TestDatabaseURL = "postgresql://postgres:postgres@localhost:5455/pets_next_door_api_test?sslmode=disable"