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 šŸ˜‰

Matt Phillips00: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

Matt Phillips00: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.

Matt Phillips04:06:05

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

Matt Phillips04: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.

Matt Phillips04: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

@U0B1SDL67 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