fix: allow !/.git excludes

Author: FabianKramm

pkg/devspace/config/loader/validate.go

@@ -362,6 +362,21 @@ func validateDevContainer(path string, devContainer *latest.DevContainer, nameRe
 				}
 			}
 		}
+		for j, p := range sync.ExcludePaths {
+			if p == "" {
+				return errors.Errorf("%s.sync[%d].excludePaths[%d] is empty. This can happen if you use !path without quotes like this: '!path'", path, index, j)
+			}
+		}
+		for j, p := range sync.UploadExcludePaths {
+			if p == "" {
+				return errors.Errorf("%s.sync[%d].uploadExcludePaths[%d] is empty. This can happen if you use !path without quotes like this: '!path'", path, index, j)
+			}
+		}
+		for j, p := range sync.DownloadExcludePaths {
+			if p == "" {
+				return errors.Errorf("%s.sync[%d].downloadExcludePaths[%d] is empty. This can happen if you use !path without quotes like this: '!path'", path, index, j)
+			}
+		}
 	}
 	for index, port := range devContainer.ReversePorts {
 		if port.Port == "" {

pkg/devspace/sync/sync.go

@@ -108,13 +108,12 @@ func NewSync(ctx context.Context, localPath string, options Options) (*Sync, err
 		return nil, errors.Wrap(err, "eval symlinks")
 	}
 
-	if options.ExcludePaths == nil {
-		options.ExcludePaths = []string{}
-	}
-
 	// We exclude the sync log to prevent an endless loop in upstream
-	options.ExcludePaths = append(options.ExcludePaths, ".devspace/")
-	options.ExcludePaths = append(options.ExcludePaths, ".git/")
+	newExcludes := []string{}
+	newExcludes = append(newExcludes, ".devspace/")
+	newExcludes = append(newExcludes, ".git/")
+	newExcludes = append(newExcludes, options.ExcludePaths...)
+	options.ExcludePaths = newExcludes
 
 	// Create sync structure
 	s := &Sync{