search

enb / enb-borschik

ENB tech to run borschik
Other
13 stars 15 forks source link

Add deps targets option to borschik technology #25

Closed tormozz48 closed 9 years ago

tormozz48 commented 9 years ago

Resolved #24

@blond @sipayRT @tavriaforever Please review it.

P.S. @aliosv It is a part of your https://github.com/enb-make/enb-borschik/pull/12.

We already have rewritten borschik technology to build-flow base. This pr should fix problem enb-make/enb#219

Please send your opinion about this

sipayRT commented 9 years ago

what about tests?

tormozz48 commented 9 years ago

what about tests?

This is build-flow functionality.

blond commented 9 years ago

The use of target word to indicate source files may introduce user confusion.

I suggest to use dependantFiles option name instead of depsTargets.

blond commented 9 years ago

@tormozz48 squash commits, please.

blond commented 9 years ago

@tormozz48 and add docs to README.md