Eugene ONeill

Results 22 comments of Eugene ONeill

Thanks for reporting. Related to #117 and sass/node-sass#1192 We don't have a good fix for this yet.

Are you able to share your gulp task? As long as the `root` eyeglass option is being set correctly, it shouldn't matter how your directory is structured.

Thanks for reporting. Can you provide an example of the root relative path it generates? The fix might be as simple as changing ```js if (!mod && moduleName[0] === "@")...

I don't believe we resolve pathnames here, so you should do something like `path.join(__dirname, "my-module")`. If this doesn't work for you, you can try using the latest master of eyeglass...

This is a "bug" in gulp-sass. See https://github.com/dlmanning/gulp-sass/pull/474 for more context. gulp-sass (incorrectly) auto-magically adds the file's parent directory onto the `includePaths`. As this is not the intended behavior of...

> Wouldn't it make sense to add the root path to includePaths so all files in root can be imported correctly? This isn't how Sass/node-sass behaves by default, so we...

To summarize: `compass-import-once` allowed forcing an import to be loaded multiple times. See https://github.com/Compass/compass/tree/master/import-once#forcing-an-import Consider supporting it in `eyeglass` when import-once is enabled.

@samuelgoddard could you elaborate on the error/issue you encounter? I tried adding a test case to reproduce this issue but wasn't able to get a failure case. Here's a passing...

I haven't tried to reproduce locally yet, but just looking at the code, my guess as to what's happening is... `@import "typi"` works as expected, resolving to `node_modules/typi/scss/_typi.scss`. However, `@import...

Thanks for reporting this. #160 will fix the relative import issue so `@import "./typi/typi"` will work. However, `@import "typi/typi"` will still be an issue because it's "working as intended". We...