Fork me on GitHub
#figwheel-main
<
2021-03-05
>
mikejcusack10:03:38

Just wanted to report here that if you test out the current CLJS master it has goog.log changes, which break Figwheel Main. At the beginning it's complaining about the wrong number of arguments passed. It successfully compiled my app, including with the new npm require syntax, but when trying to recompile for hot load it hangs. And the behavior is the same without the new syntax.

👀 1
mikejcusack12:03:39

Well that's not a fix at all since the sole purpose of the new commit is to upgrade Closure. I forked the Figwheel repos and am messing with it. So far I have the log function calls fixed and now on the logger.setLevel error. I'm confused because the code is using ' at the end of vars and I've never seen that before (only single quote at the front).

raspasov13:03:57

@U01NYKKE69G single quote at the end usually is just part of the var name

raspasov13:03:01

I typically use it/read it like “some var PRIME” (it’s some math notation thing, I believe)

mikejcusack13:03:35

I'm confused because if you type foo' in a repl it fails

raspasov13:03:36

Usually means that there’s some previous version of that var, typically called just “logger”, that was modified by the application of a function

raspasov13:03:09

(let [y 1.223
      y' (int y)]
  y')

raspasov13:03:11

In this case it’s just the logger’ local, nothing special about it

mikejcusack13:03:21

Oh, ok. Never seen that before

raspasov13:03:59

Notice that init-log-level! “builds” logger and “transforms” it into logger’

mikejcusack13:03:05

So should be (glog/setLevel logger' lvl)

mikejcusack13:03:27

I fixed them

👍 1
mikejcusack13:03:28

:figwheel {:extra-deps {com.bhauman/figwheel-core {:git/url ""
                                                   :sha "fb1aa3ea465e1b34aaa0b07065b7b0bca2c931c5"}
                        com.bhauman/figwheel-main {:git/url ""
                                                   :sha "9be32e51c72cab21a71b643c350474eec3b55a72"}
                        com.bhauman/figwheel-repl {:git/url ""
                                                   :sha "b155e7631501f90921cf2a7505055fe2c937214c"}}}

mikejcusack13:03:21

Now, I set the exception to nil when figwheel wasn't working with an exception. I'm not sure if that's how Bruce would want it. But this works.

seancorfield16:03:29

Are you going to submit PRs for those fixes @U01NYKKE69G?

seancorfield16:03:08

Or are you waiting to discuss with @U064J0EFR first?

mikejcusack16:03:21

Yeah, I was just waiting for him to chat before submitting.

1
seancorfield16:03:47

(just curious since I'm "away" from cljs again right now but planning to come back and will be using Figwheel Main so this is something that would have completely confused me)

mikejcusack16:03:18

Glad to have jumped on it then. I was hoping it would be helpful. 🙂

mikejcusack16:03:59

Gotta love upstream making a breaking change 😄

seancorfield16:03:16

SemVer is broken, remember 🙂

🎯 1