Fork me on GitHub
#cursive
<
2015-09-20
>
ricardo06:09:01

@cfleming: Let me isolate in which cases it’s exactly happening.

ricardo06:09:12

@potetm @cfleming: Yeah, definitely they should have opened with it, but it wasn’t just marketing taking over writing the copy - this one is a reworked model.

ricardo06:09:03

I expect there was some echo chambering going on with the original one. ”We like pure subscriptions, couple customers have asked us about it, therefore everyone will love it”.

ricardo06:09:38

I very much like the new approach.

ricardo06:09:25

I hadn’t even considered the budget freeze issue they mention. Talking to a few corporate customers before the original announcement would probably have helped them highlight that one.

ricardo06:09:33

@cfleming: How do you feel about the permanent license being the one for when someone’s subscription started? Looks like this would extend your necessary version support window.

ricardo06:09:10

It’s not entirely clear if these “versions available” will be major releases, like they’re doing now, or just some continuous update. I expect the major release approach negates that issue.

pupeno12:09:56

How can I have IntelliJ re-load the list of “external libraries” according to project.clj?

darwin13:09:24

@pupeno: I just quit IntelliJ and open it again.

pupeno13:09:50

@darwin: yeah, I do that, but I had some things running in intellij that I didn’t want to lose.

danielcompton13:09:37

@pupeno: run Refresh Leiningen Projects

danielcompton13:09:51

I use Command+Shift+A to bring up the command box

danielcompton13:09:20

That will reload any changes to your Lein config

simax9914:09:38

When I use the refactoring extract variable on the s-expression (utils/get-current-location). Cursive asks if I want to replace all three occurrences which I accept. It then proceeds to add the let binding outside the function definition, which doesn’t seem correct. I use extract variable in other functions and it behaves as I would expect, often offering choices of where the binding should be placed. (defn ^:export init [] (dispatch [:initialise-db]) (when (or (= (utils/get-current-location) "") (= (utils/get-current-location) "#/")) (utils/set-hash! "#/employees")) (println (str "Current loc: " (utils/get-current-location))) (hook-browser-navigation!) (mount-root)) After extract variable refactoring (let [value (utils/get-current-location)] (defn ^:export init [] (dispatch [:initialise-db]) (when (or (= value "") (= value "#/")) (utils/set-hash! "#/employees")) (println (str "Current loc: " value)) (hook-browser-navigation!) (mount-root))) I’m using Cursive version: 0.1.62 on IntelliJ version 142.4859.6 on a Mac

cfleming22:09:19

@simax99: Yeah, that’s a bug due to the fact that Cursive isn’t yet aware of which forms are in an implicit do-body (the defn body, in this case). That will be coming soon.

cfleming22:09:43

Currently it has to find a single enclosing form, which it sometimes does wrong.

cfleming22:09:39

@darwin: @pupeno: There’s also a button in the Leiningen toolwindow to refresh dependencies.

cfleming23:09:01

@ricardo: Yeah, for the moment their plan is to maintain major version releases: https://devnet.jetbrains.com/thread/472538

cfleming23:09:26

So after all the drama this hopefully won’t affect me too much.

danielcompton23:09:43

@cfleming: it could mean you need to support older versions of IntelliJ for longer? Because if someone’s license lapses they drop back to the version of 1 year ago. Although you’re already supporting older versions so maybe not a big change

cfleming23:09:25

@danielcompton: Yeah, I’ll probably just have the same policy as now, i.e. two major version support

cfleming23:09:28

If someone is on an ancient IntelliJ they may need to upgrade to use Cursive, and similarly they may need to upgrade Cursive if they upgrade IntelliJ.

cfleming23:09:15

I’m hopefully going to provide a PR for IntelliJ v15 which will allow me to switch functionality based on the IntelliJ version, which should make that more tractable.

danielcompton23:09:51

PR to Jetbrains?

cfleming23:09:06

IntelliJ is OSS, after all simple_smile

cfleming23:09:15

I actually have a couple planned.

danielcompton23:09:16

Could you do anything with custom reader conditionals?

cfleming23:09:33

Maybe, but I’d rather not simple_smile

cfleming23:09:54

Firstly I probably need to switch Java code too.

cfleming23:09:35

That would be a sure way to improve the reader conditional support though.

cfleming23:09:27

It’s actually an interesting idea, I hadn’t considered that, I probably shouldn’t say no too quickly.

cfleming23:09:22

Not sure I could handle Rich telling me “I told you so”, though.

cfleming23:09:29

Actually, it wouldn’t work since I AOT

cfleming23:09:52

I’d end up AOTing multiple versions, but I still need IntelliJ support to switch between them.