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

fix: avoid recursive rebuilds on webpack 5 #532

Merged
merged 1 commit into from
Nov 15, 2020

Conversation

piotr-oles
Copy link
Collaborator

The root cause was this.fileWatcher.close() call. In this PR I create new fileWatcher for each watch call - that follows built-in NodeWatchFileSystem semantics.

Closes: #527

@piotr-oles piotr-oles added the bug label Nov 15, 2020
@piotr-oles piotr-oles self-assigned this Nov 15, 2020
@codeclimate
Copy link

codeclimate bot commented Nov 15, 2020

Code Climate has analyzed commit 0485359 and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 2

View more on Code Climate.

Copy link
Member

@johnnyreilly johnnyreilly left a comment

Choose a reason for hiding this comment

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

Nice! Is it worth making the excluded directories configurable? Trying to imagine unusual usecases..

@piotr-oles
Copy link
Collaborator Author

I will add this if someone will ask for :)

@piotr-oles piotr-oles merged commit 8cd7304 into master Nov 15, 2020
@piotr-oles piotr-oles deleted the fix/recursive-yarn-build branch November 15, 2020 15:14
@piotr-oles
Copy link
Collaborator Author

🎉 This PR is included in version 6.0.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

Updating to v6.0.0 causes Webpack to go into a loop when watching for changes
2 participants