chlorine-clover

About Chlorine for Atom and Clover for VS Code: https://atom.io/packages/chlorine and https://marketplace.visualstudio.com/items?itemName=mauricioszabo.clover
2019-09-03T00:40:58.010200Z

it looks like one can choose auto-translate on the subtitles -- not sure how good the results are....

mauricio.szabo 2019-09-03T01:58:39.010900Z

Maybe it would be better if I record an English version ;)

2019-09-03T03:27:13.011600Z

that'd be neat, but it seems like that would be a fair bit of work

jaihindhreddy 2019-09-03T12:19:30.012700Z

When I run evaluate-block or evaluate-top-block with the cursor next to a set literal, it doesn't seem to send the # character recognizing it as a map instead.

jaihindhreddy 2019-09-03T12:19:40.013100Z

Is this a known issue or am I doing something wrong?

jaihindhreddy 2019-09-03T12:21:14.013600Z

When I select the entire set and run evaluate-selection, it works as expected.

mauricio.szabo 2019-09-03T13:00:59.015800Z

It is a known issue. I'm trying to make a new way to detect the current form (it currently uses the Atom editor's API, which is not aware of sets) but so far its not working yet

mauricio.szabo 2019-09-03T14:29:12.016300Z

My best bet now is rewrite-cljs, as every other library I tried to use failed 😞

henrik 2019-09-03T15:15:10.017600Z

I guess the same applies to derefs, reader conditionals etc.?

henrik 2019-09-03T15:15:53.018100Z

(`@(future…`, #?(:clj …)

seancorfield 2019-09-03T16:39:56.018500Z

At least there's the workaround of evaluate-selection...

henrik 2019-09-04T07:44:47.050600Z

Yeah, that’s what I do for these things. To be fair, proto-repl couldn’t handle it either.

seancorfield 2019-09-03T16:40:26.019Z

(and if you're using REBL and have my atom-chlorine-setup in place, there's a corresponding inspect-selection 🙂 )

💯 1
mauricio.szabo 2019-09-03T16:50:33.020300Z

Yes, the same happens, unfortunately. I'm hoping some library can solve this, because I really don't want to write a "simple-parser" for this problem 😱

mauricio.szabo 2019-09-03T16:50:52.020700Z

(because it seems it is not be a "simple parser" at all)

mauricio.szabo 2019-09-04T22:48:21.069800Z

@jaihindhreddy I think I got a version of the parser working already 🙂

mauricio.szabo 2019-09-04T22:49:03.070Z

It's here, if you want to use any part of my code to help on your parser too: https://github.com/mauricioszabo/repl-tooling/pull/31/files

💯 1
🦜 1
2019-09-03T22:35:12.022500Z

have not used rewrite-cljs, but rewrite-clj has been great. iiuc, the former was forked from the latter and there is someone working on merging the two -- there is also #rewrite-clj where both are on-topic

mauricio.szabo 2019-09-03T23:02:36.024Z

I think I found a way to use rewite-cljs. I became confused (and dismissed rewrite-cljs on the past) because the API of both are a little bit different, so I though they didn't implement what I need.

mauricio.szabo 2019-09-03T23:31:07.025200Z

For anyone interested, this is the beginning of the fix for derefs, reader conditionals, etc. I'll need to add more tests to check for all the cases: https://github.com/mauricioszabo/repl-tooling/commit/7766267f584b1c0251b1679d675902c6877dd6f8, but I think it's a good start (at least, the code became easier to understand and all existing tests are passing :D)