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

Support other renderers like Stylus... #6

Open
dkebler opened this issue Sep 2, 2016 · 1 comment
Open

Support other renderers like Stylus... #6

dkebler opened this issue Sep 2, 2016 · 1 comment

Comments

@dkebler
Copy link

dkebler commented Sep 2, 2016

Just took a two second look at your source and wondering if your code could be "easily" altered to be more generic by supporting generating an array of paths for any "main" file type? Pass in your file type and it returns array for same. Maybe as easy as generating your glob strings instead of hard coding them?

The stylus renderer works pretty much the same, taking an array of paths http://stylus-lang.com/docs/js.html

var sassFoldersGlobStr    = '{stylesheets,sass}';
var sassLibFoldersGlobStr = '{stylesheets,sass,lib,dist,assets/{sass,stylesheets},scss}';
var sassFilesGlobStr      = '*.{sass,scss}';
var sassGemsGlobStr = path.join('gems/*', sassFoldersGlobStr);

If this doesn't involve a major rewrite I could maybe do it as a fork with your input on the best way.

@strarsis
Copy link
Owner

strarsis commented Sep 3, 2016

Yes, this is a good idea.
It would go into a new module then (and either replace this one or with a new version of this on top of it).

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