search

at-import / node-sass-import-once

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

Better default to true for most options #14

Closed macmladen closed 8 years ago

macmladen commented 9 years ago

If someone is already including this module that is because one needs including paths on by default, so I'd suggest:

var sass = require('node-sass');,
    importOnce = require('node-sass-import-once');

sass.render({
  file: scss_filename,
  importer: importOnce,
  importOnce: {
    index: false,
    css: true,
    bower: true,
    node: true
  }
});
Snugug commented 8 years ago

The only default should be importing things once. The other options (CSS, Index, Bower, Node) are all optional modules to turn on/off, and as there's a perf overhead for enabling them, I don't think they should be on by default.