Should use Vinyl objects instead of reading from the file system about gulp-progeny

[HerringtonDarkholme]

The problem is I don't have handle to Vinyl objects after parsing dependencies in files. Using readSync is more generic

[c-vetter]

In module.exports of index.js you resolve the path and forward it until it reaches parseDeps in parse.js – which is where you have your call to fs.readFileSync. You could just as easily forward the Vinyl handle instead of the path and read the contents from that.

Using readSync is more generic

That is correct. It's also unnecessary since this is a gulp plugin and gulp –and, by extension, your code– relies on Vinyl objects. Additionally you introduce a problem that gulp wants to get around in the first place. One of gulp's main advantages over grunt is that it uses stream and thereby drastically reduces file-system access, which is one of the main bottle-necks of grunt. You may want to read up on this point here: https://github.com/substack/stream-handbook#why-you-should-use-streams

[HerringtonDarkholme]

Pull request is welcome

[c-vetter]

For anybody interested, check this out: https://www.npmjs.com/package/gulp-inheritance