search

Tonkpils / snag

Automatic build tool for all your projects
MIT License
32 stars 4 forks source link

Remove exchange dependency from watcher and builder #62

Closed Tonkpils closed 8 years ago

Tonkpils commented 8 years ago

This is the outcome of our conversation. Pretty much the builder and watcher do not have the exchange as a dependency. One thing that bothered me after doing these changes was that the watcher still has this notion of when things should be triggered to build. I can't quite pin point what it is that bothers me about it but maybe if you take a look at it you'll find it.

In any case, let me know what you think and we can either scrap it or move forward with these changes.

zabawaba99 commented 8 years ago

I'll check it out in the AM.

zabawaba99 commented 8 years ago

One thing that bothered me after doing these changes was that the watcher still has this notion of when things should be triggered to build

I don't think it does. The watcher is monitoring the file system and altering anyone who cares that something has changed. The consumer is the one handling the logic.