Skip to content

Commit

Permalink
remove unnecessary includePaths adjusts now that we're setting file p…
Browse files Browse the repository at this point in the history
…roperty correctly
  • Loading branch information
eoneill committed Dec 4, 2015
1 parent 31a3218 commit 058b42c
Showing 1 changed file with 2 additions and 12 deletions.
14 changes: 2 additions & 12 deletions index.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,25 +36,15 @@ var gulpSass = function gulpSass(options, sync) {

opts = assign({}, options);
opts.data = file.contents.toString();

// we set the file path here so that libsass can correctly resolve import paths
opts.file = file.path;

// Ensure `indentedSyntax` is true if a `.sass` file
if (path.extname(file.path) === '.sass') {
opts.indentedSyntax = true;
}

// Ensure file's parent directory in the include path
if (opts.includePaths) {
if (typeof opts.includePaths === 'string') {
opts.includePaths = [opts.includePaths];
}
}
else {
opts.includePaths = [];
}

opts.includePaths.unshift(path.dirname(file.path));

// Generate Source Maps if plugin source-map present
if (file.sourceMap) {
opts.sourceMap = file.path;
Expand Down

1 comment on commit 058b42c

@mattkahl
Copy link

Choose a reason for hiding this comment

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

I got all the way down to tracking a bug related to this and then, boom, you've already got it fixed. Thanks!

Any idea on when this will make it into a release?

Please sign in to comment.