This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2018-04-02
Channels
- # beginners (29)
- # cider (41)
- # clara (1)
- # cljs-dev (17)
- # cljsrn (1)
- # clojure (158)
- # clojure-dev (2)
- # clojure-dusseldorf (2)
- # clojure-italy (8)
- # clojure-mexico (1)
- # clojure-russia (2)
- # clojure-spec (43)
- # clojure-uk (1)
- # clojurescript (44)
- # community-development (98)
- # cursive (9)
- # data-science (8)
- # datascript (4)
- # datomic (30)
- # emacs (6)
- # fulcro (11)
- # graphql (6)
- # jobs (1)
- # jobs-discuss (27)
- # lein-figwheel (5)
- # luminus (13)
- # lumo (4)
- # off-topic (28)
- # onyx (9)
- # parinfer (12)
- # perun (2)
- # portkey (5)
- # re-frame (48)
- # ring (2)
- # shadow-cljs (52)
- # spacemacs (29)
- # tools-deps (15)
- # unrepl (9)
- # vim (7)
- # yada (3)
@dnolen is this ticket still valid? I’d like to give it a try https://dev.clojure.org/jira/browse/CLJS-2472
Pretty sure it is. Will need a bunch of tests. For tests I had in mind to finally add some kind of utility function that will generated a seq of all kinds of different collections. This would allow much better test coverage and could be used in many other tests. So that utility fn should generate seqs of all coll types in core. (IndexdSeq, PAM, PHM, PTM, chunked seqs, lazy seqs etc etc...)
Sounds good. I see there are a bunch of perf related tickets from you that seem to be not hard to fix, at least from the first look. Do you know any reason it is not done yet? I’d like to dedicate some time, if it makes sense of course.
I only provide patches these days when they're requested and I know that they'd be accepted. I could certainly provide patches for most/all of my tickets. But of course, I'm also fine if others pick up the ticket.
Got it. I guess I have to ask David about each of them.
@dnolen Can you explain why it’s necessary to put compiled sources on the inputs, not on disk, for a global preprocess function work?
Looking at the process-js-modules
in closure.clj
, it looks like the existing pipeline that we have for loading, preprocessing and compiling JS modules could already work; could just add a global preprocess function into the mix, like :preprocess
, just set as a single top-level option.
Unless process-js-modules
isn’t applied to all JS modules that we’d want to go through this new function.
@jannis it’s not necessary but I just want to do it first, there’s a bunch of benefits here for testing
@jannis also if you’re finding this a bit of challenge to do you can always just assign it to me, it’s definitely high on my list of todos
there’s a bunch of things around testing, input processing, and JS modules that I want to sort out now