perf: evaluate exclude/include filter matching only if necessary

garden-io · Dec 11, 2023 · 4850451 · 4850451
1 parent 6f6b593
commit 4850451
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/core/src/vcs/git.ts b/core/src/vcs/git.ts
@@ -367,9 +367,11 @@ export class GitHandler extends VcsHandler {
         return
       }
 
-      const passesExclusionFilter = matchPath(filePath, undefined, exclude)
-      if (hasIncludes && !passesExclusionFilter) {
-        return
+      if (hasIncludes) {
+        const passesExclusionFilter = matchPath(filePath, undefined, exclude)
+        if (!passesExclusionFilter) {
+          return
+        }
       }
 
       // We push to the output array if it passes through the exclude filters.