search

adamralph / bau

The C# task runner
MIT License
152 stars 17 forks source link

Reset the invocation state of a task with its dependencies #114

Open twiernik opened 10 years ago

twiernik commented 10 years ago

I was hesitating with ReenableWithDependencies. My other guess was to create enum which would tell if depencencies should be reenabled with dependencies or not.

connects to #115

adamralph commented 10 years ago

Thanks. I'll have to give this one a little thought.

For future reference, if you have a new feature in mind it's usually best to propose it first by raising an issue. See https://github.com/bau-build/bau/blob/dev/CONTRIBUTING.md. This will help avoid wasted effort. Of course, you're free to do whatever you want in your own fork and if you just want to experiment for your own enjoyment and send a PR regardless then no problem but there are no guarantees that the change will make it in :wink:.

twiernik commented 10 years ago

Task #55 has this feature described "Can also implement a way of resetting the invocation state of the task plus all it's dependencies.". So I treated this as a continuation of this task.

adamralph commented 10 years ago

Ah, sorry, yes, you're right. I wrote that as a bit of an afterthought and didn't really think too much about the implications. I should be more careful in what I write :wink:.

I've spun it off into a separate issue https://github.com/bau-build/bau/issues/115 and put it in the backlog (https://github.com/bau-build/bau/wiki/Issue-labels#workflow) because I think it needs some thought before going ahead with it.

twiernik commented 10 years ago

ok, cool ;)