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

Adding changed files to the override method in order to support partials change #71

Open
yonidor opened this issue Feb 3, 2015 · 2 comments

Comments

@yonidor
Copy link

yonidor commented Feb 3, 2015

I'm working with a large codebase of sass files, where some files @import other partial files.
When a partial file changes, I would like to be able to compile all of its dependencies (files that @import it), using a watch and a newer:sass tasks.

I thought the override function will allow me to to the following pseudo code:

foreach changed file
   if file is partial
      get all non-changed files that import the partial file
      include these files for the task

But the list of changed files is not available in the override function.

Instead, with current implementation, i need to go over all of the non-changed files, extract all of their @imprort-ed files, and check if the partial file has been changed lately, and only then i can decide if the file should be included or not. that is very inefficient.

I wonder if it possible to pass the list of changed files to the override function.

@ghost
Copy link

ghost commented Sep 2, 2015

See if my comment here matches what you're seeing/wanting.

#37 (comment)

@tfrommen
Copy link

Hi there,

I just posted something over here, which might be of interest to you.

@killerspaz @yonidor

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

2 participants