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 recursive watch node-sass #7

Closed
wants to merge 1 commit into from
Closed

Conversation

MrSkinny
Copy link

@MrSkinny MrSkinny commented Oct 7, 2016

it appears -wr doesn't work but -w -r does. The recursive option is not in effect otherwise, as tested by adding a second scss file, importing it into index.scss, then not seeing a rewrite occur when the second file is altered.

@MrSkinny
Copy link
Author

MrSkinny commented Oct 7, 2016

Actually, I'm getting inconsistent behavior on the watch -- sometimes it rewrites if an imported file is saved, other times in needs a forced save on the index. Based on some issues raised in node-sass repo, I think this behavior is known.

@MrSkinny
Copy link
Author

MrSkinny commented Oct 9, 2016

Reference node-sass issue -- sounds like others have built a workaround using a separate watcher and concat process before running through node-sass.

@MrSkinny
Copy link
Author

MrSkinny commented Jun 6, 2017

Closing old PRs...

@MrSkinny MrSkinny closed this Jun 6, 2017
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.

1 participant