Fork me on GitHub
#chlorine-clover
<
2020-05-30
>
seancorfield18:05:06

@mauricio.szabo How close to parity with Chlorine is your VS Code plugin? Does it have the new sci-based cljs configuration/extension stuff?

mauricio.szabo18:05:27

@seancorfield not yet. I need to figure out how to add these on VSCode. In theory, it should work. The problem is mostly that VSCode does not allow me to add commands "on the fly" - they all have to be defined on package.json file. Calva adds a dropdown menu with "additonal commands", but then you can't use keymaps to evaluate code (as far as I know - still checking if there's a way). It's also not playing nice with Clojure indentation yet 😞.

mauricio.szabo18:05:37

But... I was able to add docstrings on Autocomplete! 🙂

mauricio.szabo18:05:22

There's an API that allows Atom to get these informations lazily, so it'll not slow down autocomplete as I was fearing 🙂

💯 4
seancorfield18:05:08

That's awesome! Thank you!

seancorfield18:05:29

Re: Clover/VS Code -- if you figure out how to a) add new commands from the cljs file and b) bind them to keys, I might well switch from Atom to VS Code. There are just enough annoying differences between key bindings on Atom/macOS and Atom/Windows that I'm frustrated when switching between my laptop and my desktop -- VS Code seems better in that small respect.

pez19:05:54

I think we can find a way for chlorine to use Calva's formatter code, if we think about it together some. Of course you can just copy it, but I think we can do something a bit smarter.

mauricio.szabo19:05:32

@U0ETXRFEW great! Do you think it would be easy to publish the formatter as a npm module, or something? (Is it Javascript or ClojureScript?)

pez21:05:59

It’s both. 😀 Well, ts + cljs. Let's talk about it over zoom? But tomorrow, I'm off to bed now.

mauricio.szabo21:05:00

Yes, let's try to talk tomorrow! See you then 🙂

pez14:05:30

Hello again, I'm available for a zoom, if you are.

mauricio.szabo14:05:00

Yes, I am. Just give me about 15 minutes :)

pez14:05:36

Join when you're ready 😃 https://us02web.zoom.us/j/3584342566

seancorfield23:05:10

@mauricio.szabo Watching all those commits for 0.7.3 -- looking forward to updating to that version and trying it out! 🙂

mauricio.szabo23:05:01

Just published! 🙂

mauricio.szabo23:05:52

Also, I forgot to mention on the README and docs (need to update then) - p/let is now being used to "wait for promises" instead of let

seancorfield23:05:44

Ah, that explains why nothing works for me 🙂 Will update...

mauricio.szabo23:05:07

(I hope I did not break anything in the process 😄)

seancorfield23:05:56

Everything works locally with that change. And getNamespace() works anywhere in the file now -- THANK YOU!

seancorfield23:05:24

Updated and pushed my atom-chlorine-setup repo.

mauricio.szabo23:05:49

Wow, great! Now I have lots of thing to document 😄