Enable context-keys-type rule

.golangci.yml

@@ -55,6 +55,8 @@ linters-settings:
         disabled: false
       - name: context-as-argument
         disabled: false
+      - name: context-keys-type
+        disabled: false
       - name: error-return
         disabled: false
       - name: errorf

pkg/make/builder_test.go

@@ -17,14 +17,22 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
+type contextKey string
+
+func (c contextKey) String() string {
+	return string(c)
+}
+
+const contextKeyEnv contextKey = "ENV"
+
 func helperCommandContext(ctx context.Context, command string, args ...string) (cmd *exec.Cmd) {
 	cs := []string{"-test.run=TestHelperProcess", "--", command}
 	cs = append(cs, args...)
 	cmd = exec.CommandContext(ctx, os.Args[0], cs...) //nolint:forbidigo // Fine to use exec.CommandContext in tests
 	cmd.Env = []string{"GO_WANT_HELPER_PROCESS=1"}
 
 	// Do we have an ENV key? (type assert)
-	if ctxEnv, ok := ctx.Value("ENV").([]string); ok {
+	if ctxEnv, ok := ctx.Value(contextKeyEnv).([]string); ok {
 		cmd.Env = append(cmd.Env, ctxEnv...)
 	}
 
@@ -191,7 +199,8 @@ func TestGetVersion(t *testing.T) {
 
 			ctx, cancel := context.WithCancel(context.Background())
 			defer cancel()
-			ctx = context.WithValue(ctx, "ENV", []string{fmt.Sprintf("FAKE_GIT_DESCRIBE=%s", tt.in)})
+
+			ctx = context.WithValue(ctx, contextKeyEnv, []string{fmt.Sprintf("FAKE_GIT_DESCRIBE=%s", tt.in)})
 			os.Setenv("FAKE_GIT_DESCRIBE", tt.in)
 			ver, err := b.getVersion(ctx)
 			if tt.err == true {