v3.2.0

It should be: stdlib > libs > app

.gitignore

@@ -27,3 +27,4 @@ tags
 
 /bin
 /testdata/vars/v1
+/tmp

.goreleaser.yml

@@ -8,6 +8,10 @@ build:
   goarch:
     - 386
     - amd64
+    - arm
+    - arm64
+  goarm:
+    - 6
   ignore:
     - goos: darwin
       goarch: 386

CHANGELOG.md

@@ -1,7 +1,39 @@
 # Changelog
 
+## v3.2.0
+
+- Fix the `.task` directory being created in the task directory instead of the
+  Taskfile directory
+  ([#247](https://github.com/go-task/task/issues/247)).
+- Fix a bug where dynamic variables (those declared with `sh:`) were not
+  running in the task directory when the task has a custom dir or it was
+  in an included Taskfile
+  ([#384](https://github.com/go-task/task/issues/384)).
+- The watch feature (via the `--watch` flag) got a few different bug fixes and
+  should be more stable now
+  ([#423](https://github.com/go-task/task/pull/423), [#365](https://github.com/go-task/task/issues/365)).
+
+## v3.1.0
+
+- Fix a bug when the checksum up-to-date resolution is used by a task
+  with a custom `label:` attribute
+  ([#412](https://github.com/go-task/task/issues/412)).
+- Starting from this release, we're releasing official ARMv6 and ARM64 binaries
+  for Linux
+  ([#375](https://github.com/go-task/task/issues/375), [#418](https://github.com/go-task/task/issues/418)).
+- Task now respects the order of declaration of included Taskfiles when
+  evaluating variables declaring by them
+  ([#393](https://github.com/go-task/task/issues/393)).
+- `set -e` is now automatically set on every command. This was done to fix an
+  issue where multiline string commands wouldn't really fail unless the
+  sentence was in the last line
+  ([#403](https://github.com/go-task/task/issues/403)).
+
 ## v3.0.1
 
+- Allow use as a library by moving the required packages out of the `internal`
+  directory
+  ([#358](https://github.com/go-task/task/pull/358)).
 - Do not error if a specified dotenv file does not exist
   ([#378](https://github.com/go-task/task/issues/378), [#385](https://github.com/go-task/task/pull/385)).
 - Fix panic when you have empty tasks in your Taskfile

README.md

@@ -1,10 +1,15 @@
-![Test](https://github.com/go-task/task/workflows/Test/badge.svg)
-![GoReleaser](https://github.com/go-task/task/workflows/goreleaser/badge.svg)
-![GitHub All Releases](https://img.shields.io/github/downloads/go-task/task/total)
+<div align="center">
+  <a href="https://taskfile.dev">
+    <img src="docs/Logo.png" width="200px" height="200px" />
+  </a>
 
-# Task
+  <h1>Task</h1>
 
-Task is a task runner / build tool that aims to be simpler and easier to use
-than, for example, [GNU Make](https://www.gnu.org/software/make/).
+  <p>
+    Task is a task runner / build tool that aims to be simpler and easier to use than, for example, <a href="https://www.gnu.org/software/make/">GNU Make<a>.
+  </p>
 
-See [taskfile.dev](https://taskfile.dev) for the documentation.
+  <p>
+    See <a href="https://taskfile.dev">taskfile.dev</a> for the documentation.
+  </p>
+</div>

args/args_test.go

@@ -4,10 +4,10 @@ import (
 	"fmt"
 	"testing"
 
+	"github.com/stretchr/testify/assert"
+
 	"github.com/go-task/task/v3/args"
 	"github.com/go-task/task/v3/taskfile"
-
-	"github.com/stretchr/testify/assert"
 )
 
 func TestArgsV3(t *testing.T) {

cmd/task/task.go

@@ -9,12 +9,12 @@ import (
 	"path/filepath"
 	"syscall"
 
+	"github.com/spf13/pflag"
+
 	"github.com/go-task/task/v3"
 	"github.com/go-task/task/v3/args"
 	"github.com/go-task/task/v3/internal/logger"
 	"github.com/go-task/task/v3/taskfile"
-
-	"github.com/spf13/pflag"
 )
 
 var (

docs/README.md

@@ -1,5 +1,9 @@
 # Task
 
+<div align="center">
+  <img id="logo" src="/Logo.png" height="250px" width="250px" />
+</div>
+
 Task is a task runner / build tool that aims to be simpler and easier to use
 than, for example, [GNU Make][make].
 

docs/Taskfile.yml

@@ -10,3 +10,8 @@ tasks:
     desc: Serves the documentation site locally
     cmds:
       - docsify serve .
+
+  ico:
+    desc: Generate favicon.ico from Logo.png
+    cmds:
+      - convert -background transparent "Logo.png" -define icon:auto-resize=16,24,32,48,64,72,96,128,256 "favicon.ico"

docs/index.html

@@ -8,6 +8,19 @@
     <meta name="viewport" content="width=device-width, user-scalable=no, initial-scale=1.0, maximum-scale=1.0, minimum-scale=1.0">
     <link rel="stylesheet" href="//cdn.jsdelivr.net/npm/docsify-themeable/dist/css/theme-simple.css">
     <meta name="google-site-verification" content="VGAYkbdmuaciIDGkBe-eAg9yfZg0C6ostgonbGxxOa0" />
+    <style>
+      #logo {
+        transition: all 0.7s ease;
+      }
+      #logo:hover {
+        -webkit-transform: rotateZ(360deg);
+        -ms-transform: rotateZ(360deg);
+        transform: rotateZ(360deg);
+      }
+      .app-name-link img {
+        width: 125px;
+      }
+    </style>
   </head>
   <body>
     <div id="app"></div>
@@ -15,7 +28,8 @@
       window.$docsify = {
         name: 'Task',
         repo: 'go-task/task',
-        themeColor: '#00add8',
+        logo: 'Logo.png',
+        themeColor: '#29beb0',
         loadSidebar: true,
         auto2top: true,
         maxLevel: 3,

docs/usage.md

@@ -269,16 +269,16 @@ attribute:
 version: '3'
 
 tasks:
-  main-task:
+  greet:
+    vars:
+      RECIPIENT: '{{default "World" .RECIPIENT}}'
     cmds:
-      - task: write-file
-        vars: {FILE: "hello.txt", CONTENT: "Hello!"}
-      - task: write-file
-        vars: {FILE: "world.txt", CONTENT: "World!"}
+      - echo "Hello, {{.RECIPIENT}}!"
 
-  write-file:
+  greet-pessimistically:
     cmds:
-      - echo "{{.CONTENT}}" > {{.FILE}}
+      - task: greet
+        vars: {RECIPIENT: "Cruel World"}
 ```
 
 The above syntax is also supported in `deps`.
@@ -865,7 +865,7 @@ $ task default
 ## Short task syntax
 
 Starting on Task v3, you can now write tasks with a shorter syntax if they
-have the default settings (e.g. no custom `env:`, `vars:`, `silent:` , etc):
+have the default settings (e.g. no custom `env:`, `vars:`, `desc:`, `silent:` , etc):
 
 ```yaml
 version: '3'
@@ -873,7 +873,7 @@ version: '3'
 tasks:
   build: go build -v -o ./app{{exeExt}} .
 
-  build:
+  run:
     - task: build
     - ./app{{exeExt}} -h localhost -p 8080
 ```

internal/compiler/compiler.go

@@ -8,5 +8,6 @@ import (
 // E.g. variable merger, template processing, etc.
 type Compiler interface {
 	GetVariables(t *taskfile.Task, call taskfile.Call) (*taskfile.Vars, error)
-	HandleDynamicVar(v taskfile.Var) (string, error)
+	HandleDynamicVar(v taskfile.Var, dir string) (string, error)
+	ResetCache()
 }

internal/compiler/v2/compiler_v2.go

@@ -4,6 +4,7 @@ import (
 	"bytes"
 	"context"
 	"fmt"
+	"path/filepath"
 	"strings"
 	"sync"
 
@@ -37,8 +38,20 @@ type CompilerV2 struct {
 // 4. Taskvars file variables
 // 5. Environment variables
 func (c *CompilerV2) GetVariables(t *taskfile.Task, call taskfile.Call) (*taskfile.Vars, error) {
-	vr := varResolver{c: c, vars: compiler.GetEnviron()}
+	// NOTE(@andreynering): We're manually joining these paths here because
+	// this is the raw task, not the compiled one.
+	dir := t.Dir
+	if !filepath.IsAbs(dir) {
+		dir = filepath.Join(c.Dir, dir)
+	}
+
+	vr := varResolver{
+		c:    c,
+		dir:  dir,
+		vars: compiler.GetEnviron(),
+	}
 	vr.vars.Set("TASK", taskfile.Var{Static: t.Task})
+
 	for _, vars := range []*taskfile.Vars{c.Taskvars, c.TaskfileVars, call.Vars, t.Vars} {
 		for i := 0; i < c.Expansions; i++ {
 			vr.merge(vars)
@@ -49,6 +62,7 @@ func (c *CompilerV2) GetVariables(t *taskfile.Task, call taskfile.Call) (*taskfi
 
 type varResolver struct {
 	c    *CompilerV2
+	dir  string
 	vars *taskfile.Vars
 	err  error
 }
@@ -63,7 +77,7 @@ func (vr *varResolver) merge(vars *taskfile.Vars) {
 			Static: tr.Replace(v.Static),
 			Sh:     tr.Replace(v.Sh),
 		}
-		static, err := vr.c.HandleDynamicVar(v)
+		static, err := vr.c.HandleDynamicVar(v, vr.dir)
 		if err != nil {
 			vr.err = err
 			return err
@@ -74,7 +88,7 @@ func (vr *varResolver) merge(vars *taskfile.Vars) {
 	vr.err = tr.Err()
 }
 
-func (c *CompilerV2) HandleDynamicVar(v taskfile.Var) (string, error) {
+func (c *CompilerV2) HandleDynamicVar(v taskfile.Var, dir string) (string, error) {
 	if v.Static != "" || v.Sh == "" {
 		return v.Static, nil
 	}
@@ -92,7 +106,7 @@ func (c *CompilerV2) HandleDynamicVar(v taskfile.Var) (string, error) {
 	var stdout bytes.Buffer
 	opts := &execext.RunCommandOptions{
 		Command: v.Sh,
-		Dir:     c.Dir,
+		Dir:     dir,
 		Stdout:  &stdout,
 		Stderr:  c.Logger.Stderr,
 	}
@@ -109,3 +123,11 @@ func (c *CompilerV2) HandleDynamicVar(v taskfile.Var) (string, error) {
 
 	return result, nil
 }
+
+// ResetCache clear the dymanic variables cache
+func (c *CompilerV2) ResetCache() {
+	c.muDynamicCache.Lock()
+	defer c.muDynamicCache.Unlock()
+
+	c.dynamicCache = nil
+}

internal/compiler/v3/compiler_v3.go

@@ -4,6 +4,7 @@ import (
 	"bytes"
 	"context"
 	"fmt"
+	"path/filepath"
 	"strings"
 	"sync"
 
@@ -31,6 +32,13 @@ func (c *CompilerV3) GetVariables(t *taskfile.Task, call taskfile.Call) (*taskfi
 	result := compiler.GetEnviron()
 	result.Set("TASK", taskfile.Var{Static: t.Task})
 
+	// NOTE(@andreynering): We're manually joining these paths here because
+	// this is the raw task, not the compiled one.
+	dir := t.Dir
+	if !filepath.IsAbs(dir) {
+		dir = filepath.Join(c.Dir, dir)
+	}
+
 	rangeFunc := func(k string, v taskfile.Var) error {
 		tr := templater.Templater{Vars: result, RemoveNoValue: true}
 		v = taskfile.Var{
@@ -40,7 +48,7 @@ func (c *CompilerV3) GetVariables(t *taskfile.Task, call taskfile.Call) (*taskfi
 		if err := tr.Err(); err != nil {
 			return err
 		}
-		static, err := c.HandleDynamicVar(v)
+		static, err := c.HandleDynamicVar(v, dir)
 		if err != nil {
 			return err
 		}
@@ -61,7 +69,7 @@ func (c *CompilerV3) GetVariables(t *taskfile.Task, call taskfile.Call) (*taskfi
 	return result, nil
 }
 
-func (c *CompilerV3) HandleDynamicVar(v taskfile.Var) (string, error) {
+func (c *CompilerV3) HandleDynamicVar(v taskfile.Var, dir string) (string, error) {
 	if v.Static != "" || v.Sh == "" {
 		return v.Static, nil
 	}
@@ -79,7 +87,7 @@ func (c *CompilerV3) HandleDynamicVar(v taskfile.Var) (string, error) {
 	var stdout bytes.Buffer
 	opts := &execext.RunCommandOptions{
 		Command: v.Sh,
-		Dir:     c.Dir,
+		Dir:     dir,
 		Stdout:  &stdout,
 		Stderr:  c.Logger.Stderr,
 	}
@@ -96,3 +104,11 @@ func (c *CompilerV3) HandleDynamicVar(v taskfile.Var) (string, error) {
 
 	return result, nil
 }
+
+// ResetCache clear the dymanic variables cache
+func (c *CompilerV3) ResetCache() {
+	c.muDynamicCache.Lock()
+	defer c.muDynamicCache.Unlock()
+
+	c.dynamicCache = nil
+}

internal/execext/exec.go

@@ -46,16 +46,10 @@ func RunCommand(ctx context.Context, opts *RunCommandOptions) error {
 	}
 
 	r, err := interp.New(
+		interp.Params("-e"),
 		interp.Dir(opts.Dir),
 		interp.Env(expand.ListEnviron(environ...)),
-
-		interp.OpenHandler(func(ctx context.Context, path string, flag int, perm os.FileMode) (io.ReadWriteCloser, error) {
-			if path == "/dev/null" {
-				return devNull{}, nil
-			}
-			return interp.DefaultOpenHandler()(ctx, path, flag, perm)
-		}),
-
+		interp.OpenHandler(openHandler),
 		interp.StdIO(opts.Stdin, opts.Stdout, opts.Stderr),
 	)
 	if err != nil {
@@ -86,3 +80,10 @@ func Expand(s string) (string, error) {
 	}
 	return "", nil
 }
+
+func openHandler(ctx context.Context, path string, flag int, perm os.FileMode) (io.ReadWriteCloser, error) {
+	if path == "/dev/null" {
+		return devNull{}, nil
+	}
+	return interp.DefaultOpenHandler()(ctx, path, flag, perm)
+}

internal/output/output_test.go

@@ -6,9 +6,9 @@ import (
 	"io"
 	"testing"
 
-	"github.com/go-task/task/v3/internal/output"
-
 	"github.com/stretchr/testify/assert"
+
+	"github.com/go-task/task/v3/internal/output"
 )
 
 func TestInterleaved(t *testing.T) {

internal/status/checksum.go

@@ -14,7 +14,8 @@ import (
 // Checksum validades if a task is up to date by calculating its source
 // files checksum
 type Checksum struct {
-	Dir       string
+	BaseDir   string
+	TaskDir   string
 	Task      string
 	Sources   []string
 	Generates []string
@@ -32,7 +33,7 @@ func (c *Checksum) IsUpToDate() (bool, error) {
 	data, _ := ioutil.ReadFile(checksumFile)
 	oldMd5 := strings.TrimSpace(string(data))
 
-	sources, err := globs(c.Dir, c.Sources)
+	sources, err := globs(c.TaskDir, c.Sources)
 	if err != nil {
 		return false, err
 	}
@@ -43,7 +44,7 @@ func (c *Checksum) IsUpToDate() (bool, error) {
 	}
 
 	if !c.Dry {
-		_ = os.MkdirAll(filepath.Join(c.Dir, ".task", "checksum"), 0755)
+		_ = os.MkdirAll(filepath.Join(c.BaseDir, ".task", "checksum"), 0755)
 		if err = ioutil.WriteFile(checksumFile, []byte(newMd5+"\n"), 0644); err != nil {
 			return false, err
 		}
@@ -52,7 +53,7 @@ func (c *Checksum) IsUpToDate() (bool, error) {
 	if len(c.Generates) > 0 {
 		// For each specified 'generates' field, check whether the files actually exist
 		for _, g := range c.Generates {
-			generates, err := glob(c.Dir, g)
+			generates, err := glob(c.TaskDir, g)
 			if os.IsNotExist(err) {
 				return false, nil
 			}
@@ -107,7 +108,7 @@ func (*Checksum) Kind() string {
 }
 
 func (c *Checksum) checksumFilePath() string {
-	return filepath.Join(c.Dir, ".task", "checksum", c.normalizeFilename(c.Task))
+	return filepath.Join(c.BaseDir, ".task", "checksum", c.normalizeFilename(c.Task))
 }
 
 var checksumFilenameRegexp = regexp.MustCompile("[^A-z0-9]")

internal/status/glob.go

@@ -5,9 +5,9 @@ import (
 	"path/filepath"
 	"sort"
 
-	"github.com/go-task/task/v3/internal/execext"
-
 	"github.com/mattn/go-zglob"
+
+	"github.com/go-task/task/v3/internal/execext"
 )
 
 func globs(dir string, globs []string) ([]string, error) {

internal/summary/summary_test.go

@@ -5,11 +5,11 @@ import (
 	"strings"
 	"testing"
 
+	"github.com/stretchr/testify/assert"
+
 	"github.com/go-task/task/v3/internal/logger"
 	"github.com/go-task/task/v3/internal/summary"
 	"github.com/go-task/task/v3/taskfile"
-
-	"github.com/stretchr/testify/assert"
 )
 
 func TestPrintsDependenciesIfPresent(t *testing.T) {

status.go

@@ -76,8 +76,9 @@ func (e *Executor) timestampChecker(t *taskfile.Task) status.Checker {
 
 func (e *Executor) checksumChecker(t *taskfile.Task) status.Checker {
 	return &status.Checksum{
-		Dir:       t.Dir,
-		Task:      t.Task,
+		BaseDir:   e.Dir,
+		TaskDir:   t.Dir,
+		Task:      t.Name(),
 		Sources:   t.Sources,
 		Generates: t.Generates,
 		Dry:       e.Dry,

task.go

@@ -176,7 +176,7 @@ func (e *Executor) Setup() error {
 	if v < 2.1 && e.Taskfile.Output != "" {
 		return fmt.Errorf(`task: Taskfile option "output" is only available starting on Taskfile version v2.1`)
 	}
-	if v < 2.2 && len(e.Taskfile.Includes) > 0 {
+	if v < 2.2 && e.Taskfile.Includes.Len() > 0 {
 		return fmt.Errorf(`task: Including Taskfiles is only available starting on Taskfile version v2.2`)
 	}
 	if v >= 3.0 && e.Taskfile.Expansions > 2 {
@@ -229,10 +229,14 @@ func (e *Executor) Setup() error {
 	}
 
 	if v < 3 {
-		for _, taskfile := range e.Taskfile.Includes {
+		err := e.Taskfile.Includes.Range(func(_ string, taskfile taskfile.IncludedTaskfile) error {
 			if taskfile.AdvancedImport {
 				return errors.New(`task: Import with additional parameters is only available starting on Taskfile version v3`)
 			}
+			return nil
+		})
+		if err != nil {
+			return err
 		}
 	}
 

task_test.go

@@ -11,10 +11,10 @@ import (
 	"strings"
 	"testing"
 
+	"github.com/stretchr/testify/assert"
+
 	"github.com/go-task/task/v3"
 	"github.com/go-task/task/v3/taskfile"
-
-	"github.com/stretchr/testify/assert"
 )
 
 // fileContentTest provides a basic reusable test-case for running a Taskfile
@@ -303,16 +303,15 @@ func TestPrecondition(t *testing.T) {
 }
 
 func TestGenerates(t *testing.T) {
+	const dir = "testdata/generates"
+
 	const (
 		srcTask        = "sub/src.txt"
 		relTask        = "rel.txt"
-		absTask        = "abs.txt"
+		absTask        = "sub/abs.txt"
 		fileWithSpaces = "my text file.txt"
 	)
 
-	// This test does not work with a relative dir.
-	dir, err := filepath.Abs("testdata/generates")
-	assert.NoError(t, err)
 	var srcFile = filepath.Join(dir, srcTask)
 
 	for _, task := range []string{srcTask, relTask, absTask, fileWithSpaces} {
@@ -800,6 +799,18 @@ func TestWhenDirAttributeItCreatesMissingAndRunsInThatDir(t *testing.T) {
 	_ = os.RemoveAll(toBeCreated)
 }
 
+func TestDynamicVariablesShouldRunOnTheTaskDir(t *testing.T) {
+	tt := fileContentTest{
+		Dir:       "testdata/dir/dynamic_var",
+		Target:    "default",
+		TrimSpace: false,
+		Files: map[string]string{
+			"subdirectory/dir.txt": "subdirectory\n",
+		},
+	}
+	tt.Run(t)
+}
+
 func TestDisplaysErrorOnUnsupportedVersion(t *testing.T) {
 	e := task.Executor{
 		Dir:    "testdata/version/v1",
@@ -868,3 +879,19 @@ func TestDotenvShouldAllowMissingEnv(t *testing.T) {
 	}
 	tt.Run(t)
 }
+
+func TestExitImmediately(t *testing.T) {
+	const dir = "testdata/exit_immediately"
+
+	var buff bytes.Buffer
+	e := task.Executor{
+		Dir:    dir,
+		Stdout: &buff,
+		Stderr: &buff,
+		Silent: true,
+	}
+	assert.NoError(t, e.Setup())
+
+	assert.Error(t, e.Run(context.Background(), taskfile.Call{Task: "default"}))
+	assert.Contains(t, buff.String(), `"this_should_fail": executable file not found in $PATH`)
+}

taskfile/included_taskfile.go

@@ -1,6 +1,10 @@
 package taskfile
 
-import "errors"
+import (
+	"errors"
+
+	"gopkg.in/yaml.v3"
+)
 
 var (
 	// ErrCantUnmarshalIncludedTaskfile is returned for invalid var YAML.
@@ -15,7 +19,72 @@ type IncludedTaskfile struct {
 }
 
 // IncludedTaskfiles represents information about included tasksfiles
-type IncludedTaskfiles = map[string]IncludedTaskfile
+type IncludedTaskfiles struct {
+	Keys    []string
+	Mapping map[string]IncludedTaskfile
+}
+
+// UnmarshalYAML implements the yaml.Unmarshaler interface.
+func (tfs *IncludedTaskfiles) UnmarshalYAML(node *yaml.Node) error {
+	if node.Kind != yaml.MappingNode {
+		return errors.New("task: includes is not a map")
+	}
+
+	// NOTE(@andreynering): on this style of custom unmarsheling,
+	// even number contains the keys, while odd numbers contains
+	// the values.
+	for i := 0; i < len(node.Content); i += 2 {
+		keyNode := node.Content[i]
+		valueNode := node.Content[i+1]
+
+		var v IncludedTaskfile
+		if err := valueNode.Decode(&v); err != nil {
+			return err
+		}
+		tfs.Set(keyNode.Value, v)
+	}
+	return nil
+}
+
+// Len returns the length of the map
+func (tfs *IncludedTaskfiles) Len() int {
+	if tfs == nil {
+		return 0
+	}
+	return len(tfs.Keys)
+}
+
+// Merge merges the given IncludedTaskfiles into the caller one
+func (tfs *IncludedTaskfiles) Merge(other *IncludedTaskfiles) {
+	other.Range(func(key string, value IncludedTaskfile) error {
+		tfs.Set(key, value)
+		return nil
+	})
+}
+
+// Set sets a value to a given key
+func (tfs *IncludedTaskfiles) Set(key string, includedTaskfile IncludedTaskfile) {
+	if tfs.Mapping == nil {
+		tfs.Mapping = make(map[string]IncludedTaskfile, 1)
+	}
+	if !stringSliceContains(tfs.Keys, key) {
+		tfs.Keys = append(tfs.Keys, key)
+	}
+	tfs.Mapping[key] = includedTaskfile
+}
+
+// Range allows you to loop into the included taskfiles in its right order
+func (tfs *IncludedTaskfiles) Range(yield func(key string, includedTaskfile IncludedTaskfile) error) error {
+	if tfs == nil {
+		return nil
+	}
+	for _, k := range tfs.Keys {
+		if err := yield(k, tfs.Mapping[k]); err != nil {
+			return err
+		}
+	}
+	return nil
+}
 
 // UnmarshalYAML implements yaml.Unmarshaler interface
 func (it *IncludedTaskfile) UnmarshalYAML(unmarshal func(interface{}) error) error {

taskfile/merge.go

@@ -22,11 +22,9 @@ func Merge(t1, t2 *Taskfile, namespaces ...string) error {
 	}
 
 	if t1.Includes == nil {
-		t1.Includes = make(IncludedTaskfiles)
-	}
-	for k, v := range t2.Includes {
-		t1.Includes[k] = v
+		t1.Includes = &IncludedTaskfiles{}
 	}
+	t1.Includes.Merge(t2.Includes)
 
 	if t1.Vars == nil {
 		t1.Vars = &Vars{}

taskfile/precondition_test.go

@@ -3,10 +3,10 @@ package taskfile_test
 import (
 	"testing"
 
-	"github.com/go-task/task/v3/taskfile"
-
 	"github.com/stretchr/testify/assert"
 	"gopkg.in/yaml.v3"
+
+	"github.com/go-task/task/v3/taskfile"
 )
 
 func TestPreconditionParse(t *testing.T) {

taskfile/read/taskfile.go

@@ -7,11 +7,11 @@ import (
 	"path/filepath"
 	"runtime"
 
-	"github.com/go-task/task/v3/internal/templater"
-	"github.com/go-task/task/v3/taskfile"
-
 	"github.com/joho/godotenv"
 	"gopkg.in/yaml.v3"
+
+	"github.com/go-task/task/v3/internal/templater"
+	"github.com/go-task/task/v3/taskfile"
 )
 
 var (
@@ -58,7 +58,7 @@ func Taskfile(dir string, entrypoint string) (*taskfile.Taskfile, error) {
 		}
 	}
 
-	for namespace, includedTask := range t.Includes {
+	err = t.Includes.Range(func(namespace string, includedTask taskfile.IncludedTaskfile) error {
 		if v >= 3.0 {
 			tr := templater.Templater{Vars: &taskfile.Vars{}, RemoveNoValue: true}
 			includedTask = taskfile.IncludedTaskfile{
@@ -67,7 +67,7 @@ func Taskfile(dir string, entrypoint string) (*taskfile.Taskfile, error) {
 				AdvancedImport: includedTask.AdvancedImport,
 			}
 			if err := tr.Err(); err != nil {
-				return nil, err
+				return err
 			}
 		}
 
@@ -79,21 +79,21 @@ func Taskfile(dir string, entrypoint string) (*taskfile.Taskfile, error) {
 
 		info, err := os.Stat(path)
 		if err != nil {
-			return nil, err
+			return err
 		}
 		if info.IsDir() {
 			path = filepath.Join(path, "Taskfile.yml")
 		}
 		includedTaskfile, err := readTaskfile(path)
 		if err != nil {
-			return nil, err
+			return err
 		}
-		if len(includedTaskfile.Includes) > 0 {
-			return nil, ErrIncludedTaskfilesCantHaveIncludes
+		if includedTaskfile.Includes.Len() > 0 {
+			return ErrIncludedTaskfilesCantHaveIncludes
 		}
 
 		if v >= 3.0 && len(includedTaskfile.Dotenv) > 0 {
-			return nil, ErrIncludedTaskfilesCantHaveDotenvs
+			return ErrIncludedTaskfilesCantHaveDotenvs
 		}
 
 		if includedTask.AdvancedImport {
@@ -105,8 +105,12 @@ func Taskfile(dir string, entrypoint string) (*taskfile.Taskfile, error) {
 		}
 
 		if err = taskfile.Merge(t, includedTaskfile, namespace); err != nil {
-			return nil, err
+			return err
 		}
+		return nil
+	})
+	if err != nil {
+		return nil, err
 	}
 
 	if v < 3.0 {

taskfile/read/taskvars.go

@@ -6,9 +6,9 @@ import (
 	"path/filepath"
 	"runtime"
 
-	"github.com/go-task/task/v3/taskfile"
-
 	"gopkg.in/yaml.v3"
+
+	"github.com/go-task/task/v3/taskfile"
 )
 
 // Taskvars reads a Taskvars for a given directory

taskfile/slice.go

@@ -0,0 +1,10 @@
+package taskfile
+
+func stringSliceContains(s []string, str string) bool {
+	for _, v := range s {
+		if v == str {
+			return true
+		}
+	}
+	return false
+}

taskfile/taskfile.go

@@ -11,7 +11,7 @@ type Taskfile struct {
 	Expansions int
 	Output     string
 	Method     string
-	Includes   IncludedTaskfiles
+	Includes   *IncludedTaskfiles
 	Vars       *Vars
 	Env        *Vars
 	Tasks      Tasks
@@ -26,7 +26,7 @@ func (tf *Taskfile) UnmarshalYAML(unmarshal func(interface{}) error) error {
 		Expansions int
 		Output     string
 		Method     string
-		Includes   IncludedTaskfiles
+		Includes   *IncludedTaskfiles
 		Vars       *Vars
 		Env        *Vars
 		Tasks      Tasks

taskfile/taskfile_test.go

@@ -3,10 +3,10 @@ package taskfile_test
 import (
 	"testing"
 
-	"github.com/go-task/task/v3/taskfile"
-
 	"github.com/stretchr/testify/assert"
 	"gopkg.in/yaml.v3"
+
+	"github.com/go-task/task/v3/taskfile"
 )
 
 func TestCmdParse(t *testing.T) {

taskfile/var.go

@@ -53,21 +53,12 @@ func (vs *Vars) Set(key string, value Var) {
 	if vs.Mapping == nil {
 		vs.Mapping = make(map[string]Var, 1)
 	}
-	if !strSliceContains(vs.Keys, key) {
+	if !stringSliceContains(vs.Keys, key) {
 		vs.Keys = append(vs.Keys, key)
 	}
 	vs.Mapping[key] = value
 }
 
-func strSliceContains(s []string, str string) bool {
-	for _, v := range s {
-		if v == str {
-			return true
-		}
-	}
-	return false
-}
-
 // Range allows you to loop into the vars in its right order
 func (vs *Vars) Range(yield func(key string, value Var) error) error {
 	if vs == nil {

testdata/dir/dynamic_var/.gitignore

@@ -0,0 +1 @@
+subdirectory/dir.txt

testdata/dir/dynamic_var/Taskfile.yml

@@ -0,0 +1,11 @@
+version: '3'
+
+tasks:
+  default:
+    cmds:
+      - echo '{{.FOLDER}}' > dir.txt
+    dir: subdirectory
+    vars:
+      FOLDER:
+        sh: basename $(pwd)
+    silent: true

testdata/dir/dynamic_var/subdirectory/dir.txt

@@ -0,0 +1 @@
+subdirectory

testdata/exit_immediately/Taskfile.yml

@@ -0,0 +1,6 @@
+version: '3'
+
+tasks:
+  default: |
+    this_should_fail
+    echo "This shoudn't be print"

testdata/generates/Taskfile.yml

@@ -4,7 +4,7 @@ vars:
   BUILD_DIR: $pwd
 
 tasks:
-  abs.txt:
+  sub/abs.txt:
     desc: generates dest file based on absolute paths
     deps:
       - sub/src.txt

variables.go

@@ -60,7 +60,7 @@ func (e *Executor) CompiledTask(call taskfile.Call) (*taskfile.Task, error) {
 	new.Env.Merge(r.ReplaceVars(e.Taskfile.Env))
 	new.Env.Merge(r.ReplaceVars(origTask.Env))
 	err = new.Env.Range(func(k string, v taskfile.Var) error {
-		static, err := e.Compiler.HandleDynamicVar(v)
+		static, err := e.Compiler.HandleDynamicVar(v, new.Dir)
 		if err != nil {
 			return err
 		}

watch.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"os"
 	"os/signal"
+	"path/filepath"
 	"strings"
 	"syscall"
 	"time"
@@ -14,10 +15,7 @@ import (
 	"github.com/radovskyb/watcher"
 )
 
-var watchIgnoredDirs = []string{
-	".git",
-	"node_modules",
-}
+const watchInterval = 5 * time.Second
 
 // watchTasks start watching the given tasks
 func (e *Executor) watchTasks(calls ...taskfile.Call) error {
@@ -40,9 +38,6 @@ func (e *Executor) watchTasks(calls ...taskfile.Call) error {
 	w := watcher.New()
 	defer w.Close()
 	w.SetMaxEvents(1)
-	if err := w.Ignore(watchIgnoredDirs...); err != nil {
-		return err
-	}
 
 	closeOnInterrupt(w)
 
@@ -54,6 +49,9 @@ func (e *Executor) watchTasks(calls ...taskfile.Call) error {
 
 				cancel()
 				ctx, cancel = context.WithCancel(context.Background())
+
+				e.Compiler.ResetCache()
+
 				for _, c := range calls {
 					c := c
 					go func() {
@@ -65,9 +63,6 @@ func (e *Executor) watchTasks(calls ...taskfile.Call) error {
 			case err := <-w.Error:
 				switch err {
 				case watcher.ErrWatchedFileDeleted:
-					go func() {
-						w.TriggerEvent(watcher.Remove, nil)
-					}()
 				default:
 					e.Logger.Errf(logger.Red, "%v", err)
 				}
@@ -79,16 +74,16 @@ func (e *Executor) watchTasks(calls ...taskfile.Call) error {
 	}()
 
 	go func() {
-		// re-register each second because we can have new files
+		// re-register every 5 seconds because we can have new files, but this process is expensive to run
 		for {
 			if err := e.registerWatchedFiles(w, calls...); err != nil {
 				e.Logger.Errf(logger.Red, "%v", err)
 			}
-			time.Sleep(time.Second)
+			time.Sleep(watchInterval)
 		}
 	}()
 
-	return w.Start(time.Second)
+	return w.Start(watchInterval)
 }
 
 func isContextError(err error) bool {
@@ -109,16 +104,7 @@ func closeOnInterrupt(w *watcher.Watcher) {
 }
 
 func (e *Executor) registerWatchedFiles(w *watcher.Watcher, calls ...taskfile.Call) error {
-	oldWatchedFiles := make(map[string]struct{})
-	for f := range w.WatchedFiles() {
-		oldWatchedFiles[f] = struct{}{}
-	}
-
-	for f := range oldWatchedFiles {
-		if err := w.Remove(f); err != nil {
-			return err
-		}
-	}
+	watchedFiles := w.WatchedFiles()
 
 	var registerTaskFiles func(taskfile.Call) error
 	registerTaskFiles = func(c taskfile.Call) error {
@@ -146,12 +132,20 @@ func (e *Executor) registerWatchedFiles(w *watcher.Watcher, calls ...taskfile.Ca
 				return err
 			}
 			for _, f := range files {
-				if _, ok := oldWatchedFiles[f]; ok {
-					continue
-				}
-				if err := w.Add(f); err != nil {
+				absFile, err := filepath.Abs(f)
+				if err != nil {
 					return err
 				}
+				if shouldIgnoreFile(absFile) {
+					continue
+				}
+				if _, ok := watchedFiles[absFile]; ok {
+					continue
+				}
+				if err := w.Add(absFile); err != nil {
+					return err
+				}
+				e.Logger.VerboseOutf(logger.Green, "task: watching new file: %v", absFile)
 			}
 		}
 		return nil
@@ -164,3 +158,7 @@ func (e *Executor) registerWatchedFiles(w *watcher.Watcher, calls ...taskfile.Ca
 	}
 	return nil
 }
+
+func shouldIgnoreFile(path string) bool {
+	return strings.Contains(path, "/.git") || strings.Contains(path, "/.task") || strings.Contains(path, "/node_modules")
+}