Skip to content

fix: do not followSymlinks when watching for file changes#651

Merged
carmenberndt merged 2 commits intomasterfrom
fix_chokidar
Dec 9, 2020
Merged

fix: do not followSymlinks when watching for file changes#651
carmenberndt merged 2 commits intomasterfrom
fix_chokidar

Conversation

@carmenberndt
Copy link
Copy Markdown
Contributor

@carmenberndt carmenberndt commented Dec 9, 2020

fixes #604

In yarn workspaces packages are symlinked, so chokidar ends up watching all file changes. Setting followSymlinks: false, only the symlinks themselves will be watched for changes instead of following the link references and bubbling events through the link's path.

@carmenberndt carmenberndt added this to the 2.14.0 milestone Dec 9, 2020
@carmenberndt carmenberndt merged commit a92986b into master Dec 9, 2020
@carmenberndt carmenberndt deleted the fix_chokidar branch December 9, 2020 15:33
@bkrausz
Copy link
Copy Markdown

bkrausz commented Dec 9, 2020

Thanks for the fix @carmenberndt! Out of curiosity, shouldn't .watch only be watching that one .prisma file, even with symlinks on? Seems surprising that having symlinks in node_modules would cause extra events.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

While Prisma extension is enabled, typescript server keeps crashing on each save

3 participants