-
Notifications
You must be signed in to change notification settings - Fork 381
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Pass along sass's stats
object
#808
base: master
Are you sure you want to change the base?
Conversation
The `stats` object that's returned from the sass compiler contains some interesting info, particularly `includedFiles` which can be used to develop smarter watchers. Related to dlmanning#279, fixed dlmanning#428
I named the property |
What can we do to get this merged? This would definitely help in creating smarter watchers. |
I think if you extended the tests to cover this, it should help to land this. Also, this should probably be documented? /CC @xzyfer |
@WraithKenny are you up for this? |
Honestly, I've never quite figured out extending tests; I have no idea what I'm doing with regards to that. If anyone wants to adopt this issue/patch, you are more than welcome! |
You'd need to rebase the branch first and then add 2 tests for the new property in https://github.com/dlmanning/gulp-sass/blob/master/test/main.js |
@XhmikosR @WraithKenny I’ve created new PR with tests |
I don't have push rights, so @xzyfer will need to take care of it :) |
The
stats
object that's returned from the sass compiler contains some interesting info, particularlyincludedFiles
which can be used to develop smarter watchers. Related to #279, fixed #428