Skip to content

Conversation

@danic89
Copy link
Contributor

@danic89 danic89 commented Jan 4, 2014

I've added a possibility to define patterns for the sources included in html, scss, less files.

@jwvdiermen
Copy link
Owner

Looks good, but am I missing something? The function resolveTemplates doesn't seem to be used anywhere.

@danic89
Copy link
Contributor Author

danic89 commented Jan 5, 2014

For the fork I created new repo and didn't move local changes from my "node_modules". Now it seems to be ok

@jwvdiermen
Copy link
Owner

I think this is a good option to add, but it should be simplified. Can you check and test if to following does work for you? (NOTE: I changed the option template to templates) See commit: 635fe56

If you like, you can apply these changes on your fork and pull-request them so you'll still be visible as contributor. :)

@danic89
Copy link
Contributor Author

danic89 commented Jan 5, 2014

Great minification :). I've tested Your changes in my project and it works properly

jwvdiermen added a commit that referenced this pull request Jan 5, 2014
Added possibility to redefine default templates
@jwvdiermen jwvdiermen merged commit b105123 into jwvdiermen:master Jan 5, 2014
jwvdiermen pushed a commit that referenced this pull request May 24, 2015
support for options.flatten is added
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.

3 participants