Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Changed to escaped double-quotes to fix browser-sync #161

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

Merged
merged 1 commit into from
Sep 7, 2017
Merged

Changed to escaped double-quotes to fix browser-sync #161

merged 1 commit into from
Sep 7, 2017

Conversation

joshua-scott
Copy link
Contributor

The 'files' argument was being passed to browser-sync inside single quotes. These aren't recognised when the script runs, so hot-reloading was only listening for changes on the html file.
Switching to escaped double quotes fixes this.

The 'files' argument was being passed to browser-sync inside single quotes. These aren't recognised when the script runs, so hot-reloading was only listening for changes on the html file.
Switching to escaped double quotes fixes this.
@wesbos
Copy link
Owner

wesbos commented Sep 7, 2017

What OS are you on? The reason this is escaped is bc windows users can't have single quotes

@joshua-scott
Copy link
Contributor Author

What OS are you on? The reason this is escaped is bc windows users can't have single quotes

Windows 10. To clarify: the repo currently uses single quotes. I'm changing them to double quotes so it works for us Windows users too :)

@wesbos
Copy link
Owner

wesbos commented Sep 7, 2017

ohhh haha - I totally mixed up repos. Thanks :)

@wesbos wesbos merged commit 41ad4cb into wesbos:master Sep 7, 2017
EmmDguez pushed a commit to EmmDguez/JavaScript30 that referenced this pull request Jun 10, 2018
Changed to escaped double-quotes to fix browser-sync
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.

2 participants