marksmccann / node-sass-extra

A drop-in replacement for node-sass' Node API that adds support for globs, promises and more.
MIT License
2 stars 1 forks source link

Establish core architecture for the module #3

Closed phillipluther closed 5 years ago

phillipluther commented 5 years ago

This effort will produce the core node-sass-extra module which can be extended with feature functionality and provide a baseline to start developing against.

Initial thought - it might be easier to get rolling to do it all in a single file and start to chop it up if/when it gets unmanageable. Especially since the script is going to contain all the functionality it needs and isn't run in the browser, so modularity is only essential for sanity/organization.

marksmccann commented 5 years ago

I like that idea. If it can be one file, I would prefer that. Also, won't we ultimately need two core files; one for the API and the other for the CLI?

phillipluther commented 5 years ago

Yeah, you're absolutely right about the 2 files accounting for the CLI. I think I meant one "do all the stuff" file, and then the CLI file registers the terminal command, parses args, and just routes to the core functionality.

Good clarification.

phillipluther commented 5 years ago

Closing; added to initial-release branch by https://github.com/marksmccann/node-sass-extra/commit/c01585826195c37ef1fad9f031b089b47c15c275