fix: exclude other "ignored" files. (#1356)

watch.go

@@ -176,6 +176,16 @@ func (e *Executor) registerWatchedFiles(w *watcher.Watcher, calls ...taskfile.Ca
 }
 
 func shouldIgnoreFile(path string) bool {
-	return strings.Contains(path, "/.git") || strings.Contains(path, "/.hg") ||
+	ignorePaths := []string{
-		strings.Contains(path, "/.task") || strings.Contains(path, "/node_modules")
+		"/.task",
+		"/.git",
+		"/.hg",
+		"/.node_modules",
+	}
+	for _, p := range ignorePaths {
+		if strings.Contains(path, fmt.Sprintf("%s/", p)) || strings.HasSuffix(path, p) {
+			return true
+		}
+	}
+	return false
 }

watch_test.go

@@ -6,6 +6,7 @@ package task_test
 import (
 	"bytes"
 	"context"
+	"fmt"
 	"os"
 	"strings"
 	"testing"
@@ -79,3 +80,21 @@ Hello, World!
 	err = os.RemoveAll(filepathext.SmartJoin(dir, "src"))
 	require.NoError(t, err)
 }
+
+func TestShouldIgnoreFile(t *testing.T) {
+	tt := []struct {
+		path   string
+		expect bool
+	}{
+		{"/.git/hooks", true},
+		{"/.github/workflows/build.yaml", false},
+	}
+
+	for k, ct := range tt {
+		ct := ct
+		t.Run(fmt.Sprintf("ignore - %d", k), func(t *testing.T) {
+			t.Parallel()
+			require.Equal(t, shouldIgnoreFile(ct.path), ct.expect)
+		})
+	}
+}