-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Module errors out on gulp build. #9
Comments
Are you able to show me the I'll take a look into why this might be. |
Sure thing. Thank you for you quick response too, I think this plugin is fantastic! /*
For:
Author:
Date:
*/
/*!
Project Name
*/
// Normalize.css
@import "normalize";
// @import "../../bower_components/susy/sass/susy";
@import "susy/susy";
// Mixins, helpers & variables (@font-face fonts are in variables)
@import "variables";
@import "mixins";
@import "helpers";
// Animations
@import "anim";
// Base/defaults (html, body, typography, headings, lists, etc)
@import "base";
// Layout, grid, header, footer
@import "layout";
// Modules (could be also used for page specific styles)
@import "modules";
// Others... |
Hey @furzeface, it seems like the problem is a conflict with Susy. Commenting out Susy makes it work again. I'm not exactly sure if this still has to do with combine-mq or not. I guess combine-mq doesn't support Susy, maybe? Thanks. |
Hi, just wanted to comment and say that this definitely is not related to Susy, but rather to how the mixins or something else is written. Okay, so in this case it's related in the way Susy built some component. Additionally, I was getting the same error, and it was resulting from a mixin I was using. https://github.com/danielguillan/modernizr-mixin ... I tried looking at the mixin to figure out where it may have been breaking the output, but no luck. However, I recently updated to gulp sass 2.0.0 ... the libsass version, and that seemed to have fixed the output. Hope that helps in some way. |
Oh right, cheers guys for letting us know @besimhu @JimmyMultani - I'll investigate further into this 👍 |
I've been trying to incorporate this plugin into my gulp build, but it keeps erroring out when I attempt to run gulp. This is what I get:
This is the task that's failing:
If I comment out the combineMq() call, the task runs fine.
Any idea what's going on? Thanks.
The text was updated successfully, but these errors were encountered: