joyride

Quentin Le Guennec 2023-02-10T09:34:04.865899Z

Is there any news about rewrite-clj for joyride?

pez 2023-02-10T09:34:55.061439Z

Not other than that Joyride is better prepared for including rewrite-clj since a while.

Quentin Le Guennec 2023-02-10T09:35:15.970259Z

cool, great news.

Quentin Le Guennec 2023-02-10T09:35:35.725379Z

and great job

pez 2023-02-10T09:38:06.303779Z

I think @borkdude might want us to solve the lazy-require of dependencies before we include rewrite-clj, but I might be wrong about that.

borkdude 2023-02-10T09:38:41.562689Z

@pez Ah yes

borkdude 2023-02-10T09:38:59.568919Z

What we have to do here is create a module so shadow will compile a different .js file

borkdude 2023-02-10T09:39:22.980759Z

and then we require (in Node.js) that via the load-fn when people require rewrite-clj

borkdude 2023-02-10T09:39:32.654609Z

I can take a look at this today maybe.... maybe...

šŸ¤ž 1
Quentin Le Guennec 2023-02-10T09:40:23.009619Z

I’m curious as to why rewrite-clj would be different than any other clojurescript dependency

pez 2023-02-10T09:41:41.398759Z

It's not.

borkdude 2023-02-10T09:42:02.828199Z

It's not but so far we haven't included many CLJS dependencies yet and lazy loading is better for startup time

Quentin Le Guennec 2023-02-10T09:42:21.540479Z

Oh, okay

pez 2023-02-10T09:43:43.160599Z

Joyride is, and needs to be, very quick to start.

šŸ‘ 1
Quentin Le Guennec 2023-02-10T09:45:27.504669Z

Makes sense

pez 2023-02-10T09:55:57.949809Z

@borkdude would it make sense for joyride to be able to read bb.edn config and figure out dependencies that way?

borkdude 2023-02-10T10:17:59.412159Z

that's an entirely different issue

borkdude 2023-02-10T10:18:22.966589Z

rewrite-clj isn't compatible with joyride/SCI from source so that wouldn't help

pez 2023-02-10T10:19:53.085779Z

But would it be a thing for compatible libraries?

borkdude 2023-02-10T10:25:38.182749Z

yes, but this is a different topic right?

borkdude 2023-02-10T10:26:03.314409Z

last time we talked about it we also considered using npm for this

borkdude 2023-02-10T10:26:18.105379Z

since those scripts may depend on other npm libs

pez 2023-02-10T10:26:48.063529Z

Yes, probably a different topic. Just started to think about it.

pez 2023-02-10T10:27:16.217619Z

The Discussions on the repo might be a good place to iron it out.