Fork me on GitHub
#shadow-cljs
<
2018-08-18
>
richiardiandrea16:08:19

I found I need this when setting up custom defmethods for testing - now I require one namespace in each and every test. What does Thomas think about it?

thheller16:08:28

that seems like trying really hard to avoid having multiple independent builds?

thheller16:08:24

ie. in shadow-cljs you'd have a app, devcards and tests build instead

thheller16:08:10

and if you need to inject extra stuff into a dev build you have :preloads?

richiardiandrea16:08:37

@thheller uhm yep but I remember that we discussed about preload in node-test once and it was not working? I will try again though because I cannot recall now

thheller16:08:14

didn't I fix that? thought I did

thheller16:08:05

maybe still missing for :node-test though?

richiardiandrea16:08:20

Uhm will take a look

richiardiandrea16:08:35

Seems indeed missing, I will add it

thheller16:08:19

@richiardiandrea already added it. wil probably make a release with it later

thheller16:08:26

finishing up something else first though

richiardiandrea16:08:51

Ah ah ok cool thank you! Was about to.

richiardiandrea18:08:17

@thheller I noticed that if I am in a cider REPL and do (require '[cljs.repl :refer [doc]]) then (doc map) the output of that goes in the terminal and not in the REPL...might be a CIDER issue but do you have any hint on that?

thheller20:08:44

hmm doc probably prints during compilation. so it prints wherever the compilation was started. did you start the watch or whatever per CLI and then connected the REPL?

richiardiandrea22:08:21

And you are probably very right in saying that it prints during compilation, I seems to recall something like that when I was writing stuff in lumo

thheller22:08:38

hmm no its printed on the client

thheller22:08:45

but node-repl doesn't capture that yet

richiardiandrea22:08:07

ok cool thank you for opening the issue

thheller22:08:29

@richiardiandrea just pushed 2.6.1 with :preloads support for :node-test

👍 4
richiardiandrea22:08:00

@thheller thank you trying that 😉

richiardiandrea23:08:59

seems to work like a charm