Fork me on GitHub
#editors
<
2018-08-07
>
pez10:08:07

No, I can't spot was is particular. But my computer is weak and has very little RAM so maybe it is just one JVM process too much. Will try with only that one and whatever I need for testing the transport-fn.

Marc O'Morain12:08:50

@bozhidar Thanks for all the work on cider-nrepl :thumbsup: I've been fleshing out the ClojureVSCode extension recently to add more cider-nrepl support - one difficulty I'm having to find a spec for what the result formats are from the different cider-nrepl middleware. Is there a doc somewhere describing the keys and values? For example, I've been writing a type definition for the result of running some tests: https://github.com/avli/clojureVSCode/blob/bc14a2b99c710c8fcb5d2c9212e98f06c44acc06/src/clojureEval.ts#L16-L45 If there isn't such a doc, I'd like to make a PR to add one - what's the right place to put it?

bozhidar15:08:53

@marc-omorain Great point! So, this has to be generated automatically from the middleware descriptors like this doc https://github.com/nrepl/nREPL/blob/master/doc/ops.md

bozhidar15:08:46

There’s a bit of logic in nREPL to dump the output of the describe op as markdown and I think we should just add a command line interface for it.

bozhidar15:08:23

When cider-nrepl is loaded this would also-output the description of all its ops.

dominicm16:08:28

I wouldn't be thinking of a lein task, just a main.

richiardiandrea16:08:31

For @pez @bozhidar this is a serious cider contender. Not Clojure though 😉 maybe Calva one day? https://mobile.twitter.com/jaredforsyth/status/1026706210043490304

dominicm16:08:34

then doing lein run …

dominicm16:08:46

or clj -m gen.main …

👍 4
dominicm16:08:25

Oh, no, I realized this, yeah

dominicm16:08:26

ignore me 😄

bozhidar16:08:33

Your patch works, that’s why we deleted this. 🙂

dominicm16:08:55

the problem is that I now need to have certain dependencies loaded.

dominicm16:08:15

can I hot load my dependency hot loading system? 😄

bozhidar16:08:42

Alembic doesn’t work with Java 9+.

dominicm16:08:02

dynapath should work with this setup.

bozhidar16:08:22

Btw, I don’t get your point about the main. You want a dedicated main just for the docs generation?

dominicm16:08:45

@bozhidar yup. More universal than a lein task

bozhidar16:08:46

I planned just to add a cmd line flag about this --describe-ops

dominicm16:08:58

oh, a command line flag works too

dominicm16:08:55

@bozhidar does alembic not work period with java9+? Or it would work if it could find a DynamicClassLoader?

dominicm16:08:15

I mean, t.d.a should ship with refactor to support this functionality I suppose.

dominicm16:08:56

alembic uses dynapath, so it should work. Yeah, I think we just need to make changes in nrepl to support this.

bozhidar16:08:42

You get a runtime error with it, if I recall correctly.

bozhidar16:08:06

That was holding back a new refactor-nrepl release for a long time, so I just suggested we delete it.

dominicm16:08:12

Yeah, for sure. Makes sense.

bozhidar16:08:44

Both maintainers don’t have much time for the project, I’m trying to help them now cut finally 2.4.0, and I’m done with open-source for a very long time. 😄

😢 4
dominicm16:08:45

I think this would need a bit of jiggery to work in nrepl. Something about how nrepl is started. So a good choice to cut it for now.

bozhidar16:08:04

Maybe. Generally I want to have a hot-loading middleware straight in cider-nrepl, but who has time for all of this.

bozhidar16:08:23

Yeah, we’re just waiting to see if people will complain a lot.

dominicm16:08:47

yeah, where it goes doesn't matter. I like the feature in replant is all 🙂

bozhidar16:08:52

Did you test the latest snapshot? Supposedly we fixed it 🙂

dominicm16:08:18

I haven't yet, no. My life has significantly reduced clojure in it at the moment. I write terraform instead.

dominicm16:08:21

I miss clojure.

bozhidar16:08:46

Well, there’s open-source for you. 😉

dominicm16:08:51

conditional statements can only return strings. Not lists. Not maps. Just strings. Do you know how you return a list? You do this:

"${split(",", condition ? join(",", alistofthings) : "")}"

bozhidar16:08:54

All the Clojure I ever wrote. 😄

richiardiandrea16:08:15

@bozhidar before you quit then we need to merge my cljs-tooling patch 😃 should be good to go

richiardiandrea16:08:58

Actually tests are failing sorry...

richiardiandrea16:08:41

Renamed something and forgot to push, should be good later today

bozhidar16:08:22

Yeah, just let me know when this is ready.

richiardiandrea16:08:46

No answer from the Cljs folks on my jira, so ended up copying things over, will do sorry for the ping

bozhidar16:08:08

I’ll still be around, but starting next week I’ll be on semi-vacation for a few months and I’ll try to scale down my OSS work.

richiardiandrea16:08:48

Deserved holiday I would say ;)

bozhidar16:08:50

I’ll still take a look at patches and stuff, but I don’t plan to work on anything meaningful myself for a while.

👍 4
bozhidar16:08:29

Yeah, the last few months were pretty stressful. I certainly need some rest.

4
pez18:08:08

@richiardiandrea ocaml looks cool, even though I do not quite understand what it is I am looking at. 😃

richiardiandrea19:08:09

That is Clojure/lisp syntax compiled to Ocaml AST

richiardiandrea19:08:20

Which means lisp to native through the Ocaml compiler

pez18:08:04

@bozhidar scale down on your OSS work for the duration of the semi-vacation or also beyond that?

bozhidar21:08:14

@pez Just during the semi-vacation. 😄 Semi-vacation basically means a vacations from OSS, not a real vacation. The recent push was way too exhausting and I need to recharge my batteries a bit.

👍 8
bozhidar21:08:24

I love working on OSS projects, but juggling a ton of projects and a full-time job can really drain you.

👍 8