Fork me on GitHub
#shadow-cljs
<
2018-06-19
>
wilkerlucio12:06:22

I'm trying to run karma tests, but I'm getting this: Uncaught ReferenceError: shadow is not defined\nat : shadow is not defined\n at ev

wilkerlucio12:06:27

ideas what it can be?

wilkerlucio13:06:02

nevermind, bad karma config

wilkerlucio13:06:57

@thheller I'm playing around with karma watch + ci build watch

wilkerlucio13:06:17

they almost work, but seems like karma detects changes to the file while it's been flushed, causing syntax errors when it tries to read

wilkerlucio13:06:40

do you think would be possible to make this change more atomic? like writing to a temp file and then renaming instead of flushing directly in the final path

wilkerlucio14:06:55

I see you left a comment on source maps, is that hard to get there? I think this could be very useful to debug erros from CI

thheller14:06:30

its not hard to get there just really slow

wilkerlucio14:06:05

humm, so maybe put a flag behind it

thheller14:06:32

can take several seconds to generate a proper source map

wilkerlucio14:06:55

well, the compilation here takes about 60 seconds, adding a few might not be that bad

thheller14:06:05

during watch?

wilkerlucio14:06:23

ah, no, but that's a CI build, the watch on it was most for fun

wilkerlucio14:06:31

I usually run the tests in another way in dev

wilkerlucio15:06:51

@thheller is there a way to remove preloads from a specific module? I'm starting to work with web workers, but it's adding the preloads which I don't need there, is there a way to do that on the same build, or I need to create a new one?

currentoor23:06:48

I added this to my shadow-cljs.edn file

:dependencies [[cider/cider-nrepl "0.17.0-SNAPSHOT"]
                [refactor-nrepl "2.4.0-SNAPSHOT"]]
but trying to connect cider to the nrepl port results in
WARNING: CIDER's version (0.17.0-snapshot) does not match cider-nrepl's version (nil). Things will break!
         More information.
WARNING: clj-refactor and refactor-nrepl are out of sync.
Their versions are 2.4.0-SNAPSHOT (package: 20180420.223) and n/a, respectively.
You can mute this warning by changing cljr-suppress-middleware-warnings.