Clojurians Log v2

> Symbol Support 🎉

I'm checking the fact at Output size minimized for trivial "Hello World" programs and I see that some releases of cljs has a output way larger then other ones. I will try to write a script and build a matrix . (manual result in first response on this thread. Do not use this thread!)

$ cat src/foo/bar.cljs 
(ns foo.bar)

(.log js/console "ok")

# build: clj -Sdeps '{:deps {org.clojure/clojurescript {:mvn/version "1.10.439"}}}}' -m cljs.main --optimizations advanced --output-to 1.10.439.js -c foo.bar
# du -h master.js 1.10.597.js 1.10.520.js 1.10.439.js 1.10.63.js 
4,0K	master.js
96K	1.10.597.js
8,0K	1.10.520.js
8,0K	1.10.439.js
92K	1.10.63.js

@souenzzo let's not use threads in this channel

@souenzzo writing a matrix for the change isn't that interesting in my opinion

it's just a test for artificially trivial programs - a very small number of users would rely on this aspect of DCE

@mfikes thanks! I might edit that to add some more rationale for the removals of the REPLs and a path for alternatives (and relevant new changes to make that easier)

@mfikes fixed CLJS-3230

@alexmiller does Clojure clone the array passed to seq? https://clojure.atlassian.net/browse/CLJS-3231

hrm yeah I was pretty sure Clojure didn't - I don't think this is a bug

Hi, I'm pretty thrilled by krell (https://github.com/vouch-opensource/krell). Looks very minimal which I like. But I have some difficulties requiring installed node modules. For example: I try to (require '@react-navigation/bottom-tabs) (also without the @) but to no avail. The doc reads "Because Krell uses the ClojureScript compiler to index node_modules, you can idiomatically require anything you've installed via yarn or npm just like any ClojureScript library." but I cannot make it work. Any help would be great. Completion on the REPL with (require ' + TAB shows many things but not the node module. Am I missing something? #noob

Yes, I already restarted Metro.

@axel that's not a valid symbol

however we changed ClojureScript a while back to support strings for the library name not just symbols

(ns foo.core (:require ["@react-navigation/bottom-tabs" :as rn-bottom-tabs])) or something like that

@axel also I'm answering Krell specific questions in #cljsrn

(require '"@react-navigation/bottom-tabs") should also work though we should probably enhance this to avoid the quoting since not really necessary here

Thanks a lot @dnolen. I will try this.