Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When I set watch for a task that uses deps from includes, then the wrong watchlist is set #1504

Open
sirenkovladd opened this issue Feb 13, 2024 · 2 comments
Labels
state: needs triage Waiting to be triaged by a maintainer.

Comments

@sirenkovladd
Copy link

example:

version: '3'

tasks:
  build:
    cmds:
      - echo subtask
    sources:
      - list.json
version: '3'

includes:
  sub:
    taskfile: ./sub
    internal: true

tasks:
  dev:
    deps: [sub:build]
    watch: true
    cmds:
      - node index.js
    method: none
    sources:
      - index.js
  • Task version: v3.34.1
  • Operating system: Darwin arm64
  • Experiments enabled: false

what i see
the sub:build command is never called, even after restarting the task
after editing json, the dev task is not reload (although when editing index.js it is reload)

@task-bot task-bot added the state: needs triage Waiting to be triaged by a maintainer. label Feb 13, 2024
@sirenkovladd
Copy link
Author

list.json has been modified and here is the run result after

task: Started watching for tasks: dev
task: Watching for changes every 5s
task: "dev" started
task: watching new file: /var/folders/36/nhtbdyy51gg2n2snnk7c7gnr0000gn/T/tmp.eXNSAjpqGg/index.js
task: "sub:build" started
task: Task "sub:build" is up to date
task: [dev] node index.js
{ q: 1 }
task: "dev" finished

@sirenkovladd
Copy link
Author

workaround

includes:
  sub:
    dir: ./sub
    taskfile: ./sub
    internal: true

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state: needs triage Waiting to be triaged by a maintainer.
Projects
None yet
Development

No branches or pull requests

2 participants