Typically we had been putting all directive dependencies in the main module (in app.js) as opposed to in the directive that uses it. I am open to changing this model but I think we should be consistent one way or the other.