Merge pull request #1 from skpr/lint

Fix lint
skpr · Nov 26, 2024 · 596934b · 596934b
2 parents 5e34a9a + 993605d
commit 596934b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/cmd/skpr-k8s-mutate-nodeselector/main.go b/cmd/skpr-k8s-mutate-nodeselector/main.go
@@ -1,3 +1,4 @@
+// Package main is the entrypoint for our program.
 package main
 
 import (
@@ -44,7 +45,7 @@ func main() {
 		Short:   "Run the mutating webhook server",
 		Long:    cmdLong,
 		Example: cmdExample,
-		RunE: func(cmd *cobra.Command, _ []string) error {
+		RunE: func(_ *cobra.Command, _ []string) error {
 			logger := slog.New(slog.NewJSONHandler(os.Stdout, &slog.HandlerOptions{}))
 
 			logger.Info("Starting server")

diff --git a/internal/mutator/handler.go b/internal/mutator/handler.go
@@ -1,3 +1,4 @@
+// Package mutator for handling mutation requests.
 package mutator
 
 import (
@@ -17,6 +18,7 @@ import (
 // Annotation used to declare where a Pod should be scheduled.
 const annotation = "k8s-mutate-nodeselector.skpr.io/namespace"
 
+// Handler for responding to mutation requests.
 type Handler struct {
 	logger *slog.Logger
 	client clientcorev1.NamespaceInterface

diff --git a/internal/mutator/handler_test.go b/internal/mutator/handler_test.go
@@ -26,7 +26,7 @@ type MockNamespaceClient struct {
 }
 
 // Get a mock response of namespace and error.
-func (m *MockNamespaceClient) Get(ctx context.Context, name string, opts metav1.GetOptions) (*corev1.Namespace, error) {
+func (m *MockNamespaceClient) Get(_ context.Context, _ string, _ metav1.GetOptions) (*corev1.Namespace, error) {
 	return m.Namespace, m.Error
 }