Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Unable to watch directory #1075

Closed
ghost opened this issue Aug 17, 2015 · 1 comment · Fixed by #1078
Closed

Unable to watch directory #1075

ghost opened this issue Aug 17, 2015 · 1 comment · Fixed by #1078

Comments

@ghost
Copy link

ghost commented Aug 17, 2015

Hi,

I'm having trouble watching a directory with multiple SASS files using the following command line :
.\node_modules\.bin\node-sass --sourcemap -w .\sass -o .\css

As far as I can tell, this is because at line 228 of the node-sass script :
var graphOptions = { loadPaths: options.includePath };
the extensions are not specified. I've changed the line to :
var graphOptions = { loadPaths: options.includePath, extensions : ["scss", "sass", "css"] };
and it works perfectly.

xzyfer added a commit to xzyfer/node-sass that referenced this issue Aug 17, 2015
This fixes a bug that prevented the watcher from triggering file
compilations on changes to .sass file.

Fixes sass#1075
@xzyfer
Copy link
Contributor

xzyfer commented Aug 17, 2015

Thanks for report @fsaftoiu. You're absolutely correct and I have a patch in #1078

@xzyfer xzyfer added this to the next.patch milestone Aug 17, 2015
@xzyfer xzyfer self-assigned this Aug 17, 2015
xzyfer added a commit to xzyfer/node-sass that referenced this issue Aug 17, 2015
This fixes a bug that prevented the watcher from triggering file
compilations on changes to .sass file.

Fixes sass#1075
xzyfer added a commit to xzyfer/node-sass that referenced this issue Aug 17, 2015
This fixes a bug that prevented the watcher from triggering file
compilations on changes to .sass file.

Fixes sass#1075
@xzyfer xzyfer modified the milestones: next.patch, v3.3.0 Aug 26, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants