refactor: `getCompilerTasks` broken out into multiple functions

marksmccann commented 5 years ago

To improve readability, getCompilerTasks was broken out into three logical, stand-alone functions (getOutFile, getSourceMap, getTasks). As a result, the implementation is simpler and more maintainable.
The dynamic iterators for output and outFile now support the return of directories (they simply join the resulting directory with the source file's basename).
sourceMap also now supports directories, both directly and via the dynamic iterator. It uses the outFile basename in either case.
isFile was also refactored to test if a given path is a file more generally, instead of checking for specific extensions. This was done so that it could be used for .map extensions.

closes #21

marksmccann commented 5 years ago

@phillipluther The length of getCompilerTasks was really bugging me. I think this is a much better solution. Let me know if you disagree.

phillipluther commented 5 years ago

This looks great; I'd actually started down a similar break-down path to make sync/async stuff more manageable but didn't get anything checked in.

I'll merge this over and continue from here on #13

marksmccann / node-sass-extra