search

at-import / node-sass-import-once

Eyeglass style Import Once, but for all the things!
Other
94 stars 23 forks source link

fixed error was not forwarded #36

Open AndyOGo opened 7 years ago

AndyOGo commented 7 years ago

fixes #35

As described in https://github.com/sass/node-sass#importer--v200---experimental

importers can return error and LibSass will emit that error in response. For instance:

done(new Error('doesn\'t exist!'));
// or return synchornously
return new Error('nothing to do here');
AndyOGo commented 7 years ago

@Snugug @xzyfer Hey guys. Does any one of you two has some time check the outstanding PRs? Or is there no plan to maintain this repo. To be honest I find it really useful and want to contribute, but if no admin is maintaining this, than it will die...

AndyOGo commented 7 years ago

@metaskills @JohnAlbin @misscs @codingdesigner @iamcarrico @scottkellum Does anyone of you guys have admin access to this repo? I would really like to get some bug fixes out, but it doesn't seem to be maintained anymore. I really find it very useful, though some bugs are addressed with open PR's, they don't get merged...

AndyOGo commented 7 years ago

if anyone care we forked this repo and republished it on npm as @axa-ch/node-sass-import-once https://github.com/axa-ch/node-sass-import-once

AndyOGo commented 6 years ago

@Snugug This PR is almost a year old...

Munter commented 6 years ago

I'm hitting this problem again now and am blocked on it. I'd prefer not to have to create fork

AndyOGo commented 6 years ago

@Munter I forked it ages ago... https://github.com/axa-ch/node-sass-import-once

toshi-toma commented 5 years ago

I also face this problem. Please merge this PR.