Fork me on GitHub
#cider
<
2018-06-05
>
richiardiandrea00:06:05

so I am debugging an issue with cider-cljs-repl-form

richiardiandrea00:06:14

and (eq (car entry) repl-type) basically is equal to nil

richiardiandrea00:06:27

it looks like there is not protection against nil

richiardiandrea00:06:32

will open an issue

richiardiandrea00:06:02

actually might be easier to fix it 😉

m99400:06:25

Does anyone know why CIDER 0.17 isn’t (for me anyway) font-locking :import’s, CamelCaseIdentifiers, and Java interop calls, amongst others. regression from 0.16

m99400:06:55

actually, that may be more the bailiwick of clojure-mode…

bozhidar03:06:52

@m994 That’s the result of changes in clojure-mode. It’s not really a regression, though - most of this was intentional as that font-locking wasn’t precise and was interfering with other things. In the end we decided not to treat the CamelCase indentifiers differently. Check the changelog of clojure-mode for details.

m99404:06:05

@bozhidar thanks for the update. should have looked at the changelog

m99404:06:06

I’m pretty sure the items inside the (:require …) form used to be font-locked too? no big deal, but it looks somehow wrong to me now 😉

bozhidar04:06:10

There might be some regressions. The PR dragged on for a while and frankly I didn’t test things that extensively afterwards. Feel free to file tickets if you feels something’s off.

bozhidar04:06:47

I generally agreed that we had went overboard in the font-locking regular expressions, so some simplification was needed.

bozhidar04:06:49

Now that parseclj exists we can probably leverage it to some extent to improve the font-locking, but that’s not high on my todo list.

m99404:06:27

Yes, I gathered it was an protracted process when reading the attached discussion 😉 Simplifying the regexes makes sense — so hard to maintain, and so many opportunities for hidden performance problems. Will file a ticket re the ns thing

hkjels08:06:23

So, I was running my project yesterday with jack-in-clojurescript just fine, but today it complains about missing dependency [com.cemerick/piggieback "0.2.2" :scope "test"]. I’ve been using [cider/piggieback "0.3.5"] for the last couple of weeks, so the project has not changed. Only difference I know of, is that I deleted my elpa-directory and re-installed all packages. Do any of you have any clues to what issue I’m experiencing?

jumar10:06:53

@ what kind of error are you getting? Does lein repl runs ok? Have you upgraded java recently?

hkjels10:06:32

It’s a boot project and runs without issues on the command-line

jumar10:06:15

which version of Cider do you have.

hkjels12:06:26

CIDER 0.18.0snapshot

bozhidar14:06:02

And where exactly do you get the error about the missing dep?

hkjels07:06:07

`You are missing necessary dependencies for boot-cljs-repl. Please add the following dependencies to your project: [com.cemerick/piggieback “0.2.2” :scope “test”]`

richiardiandrea19:06:30

I have a discrepancy between what I see with clojure -Stree and cider-classpath-libs

richiardiandrea19:06:46

basically clojure/clojurescript is missing

richiardiandrea19:06:59

in (cider-classpath-libs)

richiardiandrea19:06:42

and therefore I receive an error: ClojureScript is not available. ...

richiardiandrea19:06:05

using the exact clojure command to get -Stree at the shell

richiardiandrea19:06:30

ok I found a problem