Use resoucePath for custom importers

[chrisfitz]

@jhnns, this should fix custom sass importers by passing resourcePath instead of stdin into the prev argument.

Fixes #234

[jhnns]

Thx for your PR. I'm currently on vacation, so please be patient. I'll take a look in the end of August :)

Am 11.08.2016 um 00:13 schrieb Chris Fitzgerald notifications@github.com:

@jhnns, this should fix custom sass importers by passing resourcePath instead of stdin into the prev argument.

Fixes #234

You can view, comment on, or merge this pull request online at:

#267

Commit Summary

Use resoucePath for custom importers
File Changes

M index.js (8)
M test/tools/customImporter.js (3)
Patch Links:

https://github.com/jtangelder/sass-loader/pull/267.patch
https://github.com/jtangelder/sass-loader/pull/267.diff
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

[jhnns]

Awesome, that looks good. Thanks :)

[jhnns]

Shipped as 4.0.1

[Awk34]

^ lol greenkeeper

[boennemann]

Hey all,

Greenkeeper dev here. Got here via this tweet: https://twitter.com/Jhnnns/status/782528090186588160

Sorry for this "referencing spam". We trying to avoid this, by rewriting all links in our Pull Requests leading to GitHub to our own service ("urls.greenkeeper.io"), which redirects to GitHub again, but without triggering a reference.

It looks like you've used the <user>/<repo>#<issue> short form in one of your commit messages, which is causing the reference. It sure looks like we're not handling this case properly yet.

I've opened an issue on that and I hope we can close it soon greenkeeperio/greenkeeper#305

Sorry for the inconvenience.

Best,
Stephan