feat: improve watcher performance, add forceRerunTriggers option #1424
Conversation
✅ Deploy Preview for vitest-dev ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
|
What's |
"not smart" 👀 I've recently worked on a project involving dumb routers, so I thought why not 😄 Also, the name was proposed as a joke in discord :P |
Sorry for this terrible joke. 😂 If the default watch mode is "smart", maybe instead we could have a If the word
|
packages/vitest/src/node/core.ts
Outdated
| if (this.config.watchIgnore) | ||
| this.console.error(c.dim('ignore: ') + c.yellow(this.config.watchIgnore.join(comma))) | ||
| if (this.config.watchExclude) | ||
| this.console.error(c.dim('ignore: ') + c.yellow(this.config.watchExclude.join(comma))) |
There was a problem hiding this comment.
| this.console.error(c.dim('ignore: ') + c.yellow(this.config.watchExclude.join(comma))) | |
| this.console.error(c.dim('exclude: ') + c.yellow(this.config.watchExclude.join(comma))) |
There was a problem hiding this comment.
I think exclude is a bit misleading. Renamed to watch exclude
sheremet-va
changed the title
This wasn't clear watching only the changelog |
Sorry, thank you for pointing that out. Will do better next time. |
Few things to note here:
watchIgnoreis renamed towatchExcludewatchExcludenow accepts only glob patternsforceRerunTriggersoptionforceRerunTriggersaccepts glob pattern for files, that if changed, will cause a full suite rerunforceRerunTriggersrespectswatchExclude