diff --git a/internal/cmd/root.go b/internal/cmd/root.go
index c87a8ba6636..c50f672c438 100644
--- a/internal/cmd/root.go
+++ b/internal/cmd/root.go
@@ -23,7 +23,7 @@ import (
 	"go.k6.io/k6/internal/log"
 	"go.k6.io/k6/secretsource"
 
-	_ "go.k6.io/k6/internal/secretsource" //nolint:revive
+	_ "go.k6.io/k6/internal/secretsource" // import it to register internal secret sources
 )
 
 const waitLoggerCloseTimeout = time.Second * 5
diff --git a/internal/secretsource/file/file.go b/internal/secretsource/file/file.go
index 1358336425a..08ef6751b42 100644
--- a/internal/secretsource/file/file.go
+++ b/internal/secretsource/file/file.go
@@ -1,3 +1,4 @@
+// Package file implements secret source that reads the secrets from a file as key=value pairs one per line
 package file
 
 import (
diff --git a/internal/secretsource/init.go b/internal/secretsource/init.go
index 0b81d63c6a4..de150cb7591 100644
--- a/internal/secretsource/init.go
+++ b/internal/secretsource/init.go
@@ -1,7 +1,7 @@
+// Package secretsource registers all the internal secret sources when imported
 package secretsource
 
-// TODO(@mstoykov): do we want this? or do we want to have a function like createOutputs?
 import (
-	_ "go.k6.io/k6/internal/secretsource/file" //nolint:revive
-	_ "go.k6.io/k6/internal/secretsource/mock" //nolint:revive
+	_ "go.k6.io/k6/internal/secretsource/file" // import them for init
+	_ "go.k6.io/k6/internal/secretsource/mock" // import them for init
 )
diff --git a/internal/secretsource/mock/mock.go b/internal/secretsource/mock/mock.go
index 8e76e1d2034..b5a44661097 100644
--- a/internal/secretsource/mock/mock.go
+++ b/internal/secretsource/mock/mock.go
@@ -1,3 +1,4 @@
+// Package mock implements a secret source that is just taking secrets on the cli
 package mock
 
 import (