Clojurians Log v2

@ccann: yeah profiles are merged in alphabetical order 😞

https://github.com/duct-framework/core/issues/31

Incidentally, @rickmoynihan, did you look over my reply to that issue?

I did… sorry have been meaning to reply

No problem. I’ve been meaning to put together a document listing pros and cons of the approach.

I’ve added a pros/cons message. Let me know if I’ve missed out any advantages or disadvantages that you can think of.

Is it possible to conditionally include a module? Specifically I'm wanting to define a module but only in the dev profile - i.e. when in prod the module will not be created.

Not currently, but there are plans to make that a possibility. As a workaround, you can always write a module that only activates on a certain key. In your case, you could have the module look for the :duct.core/environment key. That’s added automatically by the :duct.profile/dev profile.

Thanks @weavejester. That's an interesting idea. I think I could certainly work with that although I suspect that it may not solve everything perfectly as what I also want to do is only include the dependencies for that module in leiningen "dev" profile - (i.e. in "test" scope in Maven parlance). I'll give it a go though and will report back.

For reference, there’s a discussion on reworking profiles here: https://github.com/duct-framework/core/issues/31

Thanks that worked. For completeness, here's what my module init-key now looks like:

(defmethod ig/init-key :duct.module/my-module [_ opts]
  (fn [config]
    (let [environment (::duct/environment config :production)]
      (if (= :development environment)
        (duct/merge-configs
          config
          (build-config opts))
        config))))

@weavejester replied on GH… https://github.com/duct-framework/core/issues/31#issuecomment-557634893

it’s a better place than here.

Thanks @rickmoynihan. I posted a response. I’d also like to invite anyone else who’s been using Duct for a while to take a look at that issue and tell me what they think.

@iarenaza If you have time, I’d be interested to hear from the folks at Magnet about their opinion on issue: https://github.com/duct-framework/core/issues/31