Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

Upgrade chokidar disable globbing issue 212 #227

Merged
merged 5 commits into from Apr 4, 2018
Merged

Upgrade chokidar disable globbing issue 212 #227

merged 5 commits into from Apr 4, 2018

Conversation

naturalethic
Copy link
Contributor

This fixes #212

src/cli/dev.ts Outdated
@@ -311,7 +311,8 @@ function watch_files(pattern: string, events: string[], callback: () => void) {

const watcher = chokidar.watch(pattern, {
persistent: true,
ignoreInitial: true
ignoreInitial: true,
disableGlobing: true
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be disableGlobbing?

@Rich-Harris
Copy link
Member

Thanks — am slightly curious about how this works, don't we need globbing? One of the patterns we're using is ${locations.routes()}/**/* (here)

@naturalethic
Copy link
Contributor Author

Uh, yes it should be disableGlobbing. This is what I get for submitting pull request from different machine than I tested on.

@naturalethic
Copy link
Contributor Author

Please see this issue discussion paulmillr/chokidar#699

@Rich-Harris Rich-Harris merged commit 7e263a3 into sveltejs:master Apr 4, 2018
@Rich-Harris
Copy link
Member

thanks — publishing 0.10.4

@naturalethic naturalethic deleted the upgrade-chokidar-disable-globbing-issue-212 branch April 4, 2018 17:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Exception thrown with routes/[foo]/[bar].html on Windows
2 participants