Fork me on GitHub
#calva
<
2019-06-09
>
pez07:06:55

@vlaaad and now I remembered to fix that aliases jack-in issue. V 2.0.6.

vlaaad07:06:58

I personally can't see myself going to back to using text search for finding usages, so I'll stick to cursive for now... Which is a shame, vscode/calva looks very neat otherwise

pez08:06:36

Good to know. I’d welcome a feature request on github with that. Not sure what it would take to implement, but anyway.

dominicm08:06:11

@pez refactor-nrepl has it, and I think cider is getting a weaker version of it

pez09:06:32

Thanks! I created the issue now: https://github.com/BetterThanTomorrow/calva/issues/215 Anyone with knowledge about it, please add info to that issue. (And of course, feel invited to go ahead with the implementation.)

👍 4
ericdallo15:06:42

Hi guys, first of all, I would like to congratulate the work on Calva 2.0, I've been using and it's awesome! There's a famous test framework called Midje(https://github.com/marick/Midje) and i noticed that calva-fmt is not identing the macros from this framework with 2 spaces like deftest and others. Example:

(facts "when foo meets bar"
       (is (= 2 (+ 1 1)))
where the correct(i think) should be:
(facts "when foo meets bar"
  (is (= 2 (+ 1 1))))
I would like to ask if this is a valid issue to calva-fmt, Can I open a PR for that problem?

pez16:06:44

Welcome @ericdallo ! Glad you like it! Yes, please open an issue. Or, rather, search for one first, I don't remember if it has been requested there about the ability to help Calva with formatting rules. It is often coming up in here though. And please file the issue on the Calva repo, the calva-fmt repo is not active any longer.

ericdallo16:06:55

Thank you @pez, I've searched before posting here and there's no open/closed issues related to this problem with Midje. I'll open a issue then in the Calva repo.

pez16:06:20

I don't think it is specific to Midje. What is needed is a way to feed Calva with clj-fmt config maps.

pez16:06:45

But please mention your use case in the issue.

ericdallo16:06:09

hum, I see, makes sense, I will!

ericdallo16:06:14

Thank you!

metal 4
ericdallo16:06:35

I found this open issue, but I'm not sure if is the same problem: https://github.com/BetterThanTomorrow/calva/issues/80

pez16:06:59

Yeah, there's the issue. I've been focusing on other things a while now. 😀 Chime in on that one!

metal 4