Ver Fonte

fix linting issues

Signed-off-by: Sebastiaan van Stijn <[email protected]>
Sebastiaan van Stijn há 1 mês atrás
pai
commit
89d3944837

+ 1 - 0
cmd/formatter/consts.go

@@ -22,6 +22,7 @@ const (
 	// TemplateLegacyJSON the legacy json formatting value using go template
 	TemplateLegacyJSON = "{{json.}}"
 	// PRETTY is the constant for default formats on list commands
+	//
 	// Deprecated: use TABLE
 	PRETTY = "pretty"
 	// TABLE Print output in table format with column headers (default)

+ 1 - 1
cmd/formatter/formatter_test.go

@@ -45,7 +45,7 @@ func TestPrint(t *testing.T) {
 	}
 
 	b := &bytes.Buffer{}
-	assert.NilError(t, Print(testList, PRETTY, b, func(w io.Writer) {
+	assert.NilError(t, Print(testList, TABLE, b, func(w io.Writer) {
 		for _, t := range testList {
 			_, _ = fmt.Fprintf(w, "%s\t%s\n", t.Name, t.Status)
 		}

+ 6 - 0
pkg/compose/loader.go

@@ -95,12 +95,18 @@ func (s *composeService) buildProjectOptions(options api.ProjectLoadOptions, rem
 	}
 
 	opts = append(opts,
+		// Load PWD/.env if present and no explicit --env-file has been set
 		cli.WithEnvFiles(options.EnvFiles...),
+		// read dot env file to populate project environment
 		cli.WithDotEnv,
+		// get compose file path set by COMPOSE_FILE
 		cli.WithConfigFileEnv,
+		// if none was selected, get default compose.yaml file from current dir or parent folder
 		cli.WithDefaultConfigPath,
+		// .. and then, a project directory != PWD maybe has been set so let's load .env file
 		cli.WithEnvFiles(options.EnvFiles...),
 		cli.WithDotEnv,
+		// eventually COMPOSE_PROFILES should have been set
 		cli.WithDefaultProfiles(options.Profiles...),
 		cli.WithName(options.ProjectName),
 	)

+ 6 - 6
pkg/compose/model.go

@@ -39,16 +39,16 @@ func (s *composeService) ensureModels(ctx context.Context, project *types.Projec
 		return nil
 	}
 
-	api, err := s.newModelAPI(project)
+	mdlAPI, err := s.newModelAPI(project)
 	if err != nil {
 		return err
 	}
-	defer api.Close()
-	availableModels, err := api.ListModels(ctx)
+	defer mdlAPI.Close()
+	availableModels, err := mdlAPI.ListModels(ctx)
 
 	eg, ctx := errgroup.WithContext(ctx)
 	eg.Go(func() error {
-		return api.SetModelVariables(ctx, project)
+		return mdlAPI.SetModelVariables(ctx, project)
 	})
 
 	for name, config := range project.Models {
@@ -57,12 +57,12 @@ func (s *composeService) ensureModels(ctx context.Context, project *types.Projec
 		}
 		eg.Go(func() error {
 			if !slices.Contains(availableModels, config.Model) {
-				err = api.PullModel(ctx, config, quietPull, s.events)
+				err = mdlAPI.PullModel(ctx, config, quietPull, s.events)
 				if err != nil {
 					return err
 				}
 			}
-			return api.ConfigureModel(ctx, config, s.events)
+			return mdlAPI.ConfigureModel(ctx, config, s.events)
 		})
 	}
 	return eg.Wait()

+ 0 - 1
pkg/e2e/cancel_test.go

@@ -1,5 +1,4 @@
 //go:build !windows
-// +build !windows
 
 /*
    Copyright 2020 Docker Compose CLI authors

+ 0 - 1
pkg/e2e/cascade_test.go

@@ -1,5 +1,4 @@
 //go:build !windows
-// +build !windows
 
 /*
    Copyright 2022 Docker Compose CLI authors

+ 0 - 1
pkg/e2e/container_name_test.go

@@ -1,5 +1,4 @@
 //go:build !windows
-// +build !windows
 
 /*
    Copyright 2022 Docker Compose CLI authors

+ 0 - 1
pkg/e2e/noDeps_test.go

@@ -1,5 +1,4 @@
 //go:build !windows
-// +build !windows
 
 /*
    Copyright 2022 Docker Compose CLI authors

+ 0 - 1
pkg/e2e/up_test.go

@@ -1,5 +1,4 @@
 //go:build !windows
-// +build !windows
 
 /*
    Copyright 2022 Docker Compose CLI authors

+ 0 - 1
pkg/watch/watcher_darwin.go

@@ -1,5 +1,4 @@
 //go:build darwin
-// +build darwin
 
 /*
    Copyright 2020 Docker Compose CLI authors

+ 0 - 1
pkg/watch/watcher_naive.go

@@ -1,5 +1,4 @@
 //go:build !darwin
-// +build !darwin
 
 /*
    Copyright 2020 Docker Compose CLI authors

+ 0 - 1
pkg/watch/watcher_nonwin.go

@@ -1,5 +1,4 @@
 //go:build !windows
-// +build !windows
 
 /*
    Copyright 2020 Docker Compose CLI authors

+ 0 - 1
pkg/watch/watcher_windows.go

@@ -1,5 +1,4 @@
 //go:build windows
-// +build windows
 
 /*
    Copyright 2020 Docker Compose CLI authors