My exact thoughts.

Glad you spotted that.

Modular should merely be an interceptor that adds deps it can infer.

Component should to the system-using. I'll remove that from modular, but
feel free to make those changes yourself as co-author.
On 19 Mar 2014 12:34, "Neale Swinnerton" <notifications@github.com> wrote:

> In dev/kixi/hecuba/system.clj:
>
> > +    (-> (component/system-map
> > +         :cluster (new-cluster (:cassandra-cluster cfg))
> > +         :session (new-session (:cassandra-session cfg))
> > +         :schema (new-schema)
> > +         :store (new-direct-store)
> > +         :cljs-builder (new-cljs-builder)
> > +         :web-server (new-webserver (:web-server cfg))
> > +         :bidi-ring-handler (new-bidi-ring-handler-provider)
> > +         :main-routes (new-main-routes)
> > +         :amon-api (new-amon-api "/3")
> > +         :user-api (new-user-api)
> > +         :cljs-routes (new-cljs-routes (:cljs-builder cfg))
> > +         :user-data-loader (new-user-data-loader cfg)
> > +         :csv-loader (new-csv-loader cfg))
> > +
> > +        (mod/system-using
>
> Can we do this by threading some modular fns before calling
> mod/system-using. It would be good to be explicit that modular is adding to
> stuartsierra component rather than changing the semantics. I think explicit
> is better here, even if it's a little more code.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/MastodonC/kixi.hecuba/pull/6/files#r10746635>
> .
>