Wording

Signed-off-by: Danny Kopping <[email protected]>

Author: dannykopping

envbuilder.go

@@ -140,7 +140,7 @@ type Options struct {
 	DockerfilePath string `env:"DOCKERFILE_PATH"`
 
 	// BuildContextPath can be specified when a DockerfilePath is specified outside the base WorkspaceFolder.
-	// This path MUST be a relative path since it will be joined to the WorkspaceFolder path.
+	// This path MUST be relative to the WorkspaceFolder path into which the repo is cloned.
 	BuildContextPath string `env:"BUILD_CONTEXT_PATH"`
 
 	// CacheTTLDays is the number of days to use cached layers before

integration/integration_test.go

@@ -758,7 +758,7 @@ COPY %s .`, testImageAlpine, inclFile)
 		},
 		{
 			// Dockerfile & build context are not in the same dir, but inclFile is in the default build context dir;
-			// this should allow inclFile to be copied.
+			// this should allow inclFile to be copied. This is probably not desirable though?
 			name: "different build context (default, different content roots)",
 			files: map[string]string{
 				"a/Dockerfile": dockerfile,