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

Possible performance issue #10

Closed
dylang opened this issue Nov 29, 2012 · 3 comments
Closed

Possible performance issue #10

dylang opened this issue Nov 29, 2012 · 3 comments

Comments

@dylang
Copy link

dylang commented Nov 29, 2012

It seems like we're requiring all sass files to be re-read and processed by sass every time this task runs.

When used during development with watch does this means every time a sass file is changed the entire collection needs to be re-read off the HD and re-processed by sass?

If so this seems like a potential performance hit for sites with a lot of sass code.

I'm very new to sass and don't know what the right solution is for this.

@shama
Copy link
Member

shama commented Nov 29, 2012

There is a discussion on this over at gruntjs/grunt-contrib-watch#14
(at least its the latest of many discussions on this issue)

@sindresorhus
Copy link
Member

I'm fully aware, I've mentioned it on several occasions, but it's not specific to this task. Almost every task could be sped up by only processing changed files. This is especially apparent in tasks like Sass or image minification.

For now, one thing you can do is to create separate watch targets for different parts of your app, if you have many different Sass pieces. That way only the files in each target will be processed when one is changed. Though this is only a temporary bandaid.

@tkellen
Copy link
Member

tkellen commented Dec 10, 2012

Closing this for now. This definitely won't be solved at the sass task level. This is on our radar, but it's a long way out.

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

No branches or pull requests

4 participants