Skip to content

Commit 847a2d3

Browse files
committed
Merge pull request #417 from ajschlosser/master
add relativePath property to error object for notification(s) support
2 parents d62227f + 5e8af62 commit 847a2d3

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

index.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -127,6 +127,8 @@ var gulpSass = function gulpSass(options, sync) {
127127
error.messageFormatted = message;
128128
error.message = gutil.colors.stripColor(message);
129129

130+
error.relativePath = relativePath;
131+
130132
return cb(new gutil.PluginError(
131133
PLUGIN_NAME, error
132134
));

test/main.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -148,6 +148,8 @@ describe('gulp-sass -- async compile', function() {
148148
err.message.indexOf('test/scss/error.scss').should.not.equal(-1);
149149
// Error must include line and column error occurs on
150150
err.message.indexOf('on line 2').should.not.equal(-1);
151+
// Error must include relativePath property
152+
err.relativePath.should.equal('test/scss/error.scss');
151153
done();
152154
});
153155
stream.write(errorFile);
@@ -379,6 +381,7 @@ describe('gulp-sass -- sync compile', function() {
379381

380382
stream.on('error', function(err) {
381383
err.message.indexOf('property "font" must be followed by a \':\'').should.not.equal(-1);
384+
err.relativePath.should.equal('test/scss/error.scss');
382385
done();
383386
});
384387
stream.write(errorFile);

0 commit comments

Comments
 (0)