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 for gulp watching on windows #2643

Merged
merged 3 commits into from
Jul 6, 2020
Merged

Conversation

Tusky
Copy link
Contributor

@Tusky Tusky commented Jun 28, 2020

Description

Fix for browsersync and file watchers not working correctly on windows. This uses a legacy polling method, so it registers events if the user is on windows (which I assume the windows? is meant to represent.

Fixes #2642 which I've encountered.

Rationale

There are some windows developers around, and cookiecutter-django is a great start for a project whatever OS you use.

Use case(s) / visualization(s)

Not much to visualize sadly, but I think it's a pretty simple PR to understand what it does :)

Fix for browsersync and file watchers not working correctly on windows.
@Tusky Tusky changed the title Patch 1 Fix for browsersync Jun 28, 2020
@Tusky Tusky changed the title Fix for browsersync Fix for gulp watching on windows Jun 28, 2020
Copy link
Member

@browniebroke browniebroke left a comment

Choose a reason for hiding this comment

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

LGTM (I don't have a Windows machine, so I cannot test it)

@Andrew-Chen-Wang
Copy link
Contributor

Andrew-Chen-Wang commented Jun 29, 2020

@Tusky For the CONTRIBUTOR's doc, RST hyperlinks work by having the `@Tusky`_ with the link to your GitHub. You can take a look below all the names in the table to know what I mean.

@Tusky
Copy link
Contributor Author

Tusky commented Jun 29, 2020

@Tusky For the CONTRIBUTOR's doc, RST hyperlinks work by having the @Tusky_ with the link to your GitHub. You can take a look below all the names in the table to know what I mean.

Thank you, forgot to add it at the bottom :(

@browniebroke browniebroke merged commit 22d998a into cookiecutter:master Jul 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Node instance is not registering file changes with browsersync
3 participants