Support for an expanded compass --require arg #483

Merged
merged 1 commit into from Sep 17, 2012

Conversation

Projects
None yet
3 participants
Contributor

jeffinmotion commented Sep 17, 2012

As it stands, the Gruntfile compass configuration will only let me require 1 compass plugin via the --require arg.

This change to the optsToArgs function would allow arrays in the config and properly expand them for the spawn util.

So for example:

    ...
    compass: {
      dist: {
        require: ['modular-scale', 'susy'],
    ...

would get properly expanded to:
--require modular-scale --require susy

sindresorhus merged commit 17e332e into yeoman:master Sep 17, 2012

1 check passed

default The Travis build passed
Details
Owner

sindresorhus commented Sep 17, 2012

@jeffinmotion Awesome, didn't think of that. Thanks :)

Contributor

jeffinmotion commented Sep 17, 2012

No problem!

is below the right config for compass plugins?, using suzy as an example.

 compass: {
      dist: {
        // http://compass-style.org/help/tutorials/configuration-reference/#configuration-properties
        require: ['susy'],
        options: {
          css_dir: 'temp/styles',
          sass_dir: 'app/styles',
          images_dir: 'app/images',
          javascripts_dir: 'temp/scripts',
          force: true
        }
      }
    }

doesn't seem to pickup susy mixins in my SCSS files.

Contributor

jeffinmotion commented Oct 8, 2012

@imranansari —close! --require is a compass param, so it needs to go in your compass options.

compass: {
  dist: {
    // http://compass-style.org/help/tutorials/configuration-reference/#configuration-properties
    options: {
      css_dir: 'temp/styles',
      sass_dir: 'app/styles',
      images_dir: 'app/images',
      javascripts_dir: 'temp/scripts',
      require: ['susy'],
      force: true
    }
  }
}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment