Skip to content
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

Changed to a multitask and made it work with grunt #3

Merged
merged 3 commits into from
Jun 3, 2012

Conversation

ronaldlokers
Copy link
Contributor

  • changed to a multitask so you can add multiple configurations for the task
  • made the tasks asynchronous so that it will work with grunt

Changed the task to a multitask so you can configure it for more
targets like this:

```javascript
compass: {
    dev: {
        src: 'sass',
        dest: 'dev/css',
        outputstyle: 'expanded'
      },
      dist: {
        src: 'sass',
        dest: 'dist/css',
        outputstyle: 'compressed'
      }
}
```
- made the task "asynchronous" so that it will work with `grunt`
- disabled "stderr" output since it gives falsy error messages, the
real errors come through "error"
@kahlil
Copy link
Owner

kahlil commented Jun 3, 2012

Hey Ronald! Thanks a lot! Can make a new fork and work your changes into that one? I made an npm package out of the project its now available as an npm grunt task.

@kahlil
Copy link
Owner

kahlil commented Jun 3, 2012

Thanks

kahlil added a commit that referenced this pull request Jun 3, 2012
Changed to a multitask and made it work with `grunt`
@kahlil kahlil merged commit 22ae015 into kahlil:master Jun 3, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants