Fork me on GitHub
#polylith
<
2022-06-04
>
tengstrand15:06:05

Can someone help me find a use case for issue https://github.com/polyfy/polylith/issues/230 ? I can’t see the value of using it in bricks and for the development environment we specify the paths in :aliases :dev :extra-paths .

tengstrand18:06:17

Okay. Let’s see if he finds the question in the polylith channel.

tengstrand18:06:16

btw, I’m working on issue https://github.com/polyfy/polylith/issues/222 right now, and I will add one more key stop-execution? to the test runner protocol:

(run-tests [this {:keys [class-loader color-mode eval-in-project stop-execution? is-verbose] :as opts}]))

tengstrand18:06:38

It’s an atom that will be set to true if the user presses ctrl-c.

imre18:06:50

Ah interesting!

seancorfield18:06:52

Re: alias resolution in paths - I'm afk this weekend but the idea is to be able to reduce repetition and duplication by putting common expressions in aliases and reuse that alias instead of repeating is contents (aliases are just data).

tengstrand18:06:06

Okay. When you have time @U04V70XH6 (no hurry) could you maybe update issue 222 with an example or explanation where it will be useful for the poly tool? I’m just afraid that no one will use it in a Polylith context, but maybe I just don’t see the use case of it here!

seancorfield05:06:02

My initial thought is that it won't be useful for Polylith because :paths are determined by convention mostly but, for me, it's a case of: tools.deps.alpha and the Clojure CLI supports it directly so, if Polylith is now paying attention to the actual contents of :paths, it should do it "correctly" like the CLI/t.d.a. does.

seancorfield05:06:01

I'll try to find some time this week to dig up the exact code that supports it in t.d.a and link to that from the ticket.

👍 1