Fork me on GitHub
#cider
<
2015-12-03
>
robert-stuttaford04:12:36

update lein. an older version forced an old nrepl

bozhidar09:12:26

btw, there’s a section of the readme dedicated to this simple_smile

robert-stuttaford10:12:20

congrats bozhidar and malabarba !

juhoteperi10:12:49

Is there a changelog (besides git log) for cider-nrepl?

robert-stuttaford10:12:59

it’s all in the emacs package’s changelog

robert-stuttaford10:12:35

only place i think

juhoteperi10:12:06

I'm trying to figure out if nrepl middleware has new features that vim-fireplace should support so Emacs package changelog is not very useful

juhoteperi10:12:41

Out middleware looks useful and eventually Vim plugins should use the indentation metadata

malabarba11:12:40

@juhoteperi: There's no changelog specific to cider-nrepl. Both sides of the project have their changes logged on the cider repo.

malabarba11:12:52

You can, however, checkout the git diff on the cider-nrepl project. That should tell you which files changed or are new.

malabarba11:12:53

Or check recently merged PRs. Most features in cider-nrepl are initially added as PRs (even by us)

malabarba11:12:56

Some things I can tell you off the top of my head (that are specific to cider-nrepl): - debugger changed a lot, - print-method.clj (tough you shouldn't have to do anything specific to support this) - track_state.clj - out.clj - The inspector does paginaton - the refresh middleware changed a bit too

juhoteperi12:12:22

out is interesting but I think other stuff is not used by Fireplace currently

malabarba12:12:16

I'd be interested to know if the new print-methods work in Fireplace out-of-the-box

malabarba12:12:35

I have a feeling they'll work anywhere that has cider-nrepl installed

juhoteperi12:12:42

Well, I updated my nrepl middleware and haven't noticed any problems yet.

malabarba12:12:48

but it might require manually requiring the namespace

juhoteperi12:12:01

Yeah I had to require the ns to enable new multimethod print-method

juhoteperi12:12:04

I wonder what would be the correct place to require it? In Fireplace code or in Lein/Boot profile?

malabarba12:12:59

In cider we have it in the :requires of the namespace that activates the middleware

malabarba12:12:53

although I think I just realised that's not enough ¬¬

juhoteperi13:12:50

In Fireplace there is not any code which activates the middleware, instead user needs to add it to profile plugins

sdegutis16:12:12

Is it possible to make cider-interrupt interrupt a Process running on the main thread created by ProcessBuilder?

sdegutis16:12:34

Or better yet, how do you make any Clojure code interruptible by cider-interrupt?

malabarba16:12:45

To be interruptible by cider-interrupt it needs to be running under an eval session managed by nrepl

malabarba16:12:06

@sdegutis: See, for instance, the with-interruptible-eval macro in cider/nrepl/middleware/test.clj

sdegutis16:12:06

Hmm. So then I probably just need to join my ProcessBuilder's Process to the current thread immediately after starting it.

sdegutis16:12:08

Or something.

rickmoynihan17:12:53

@bozhidar: And other cider devs -- congratulations on the 0.10.0 release... The debugger is awesome!!

rickmoynihan17:12:59

loving the overlays

malabarba17:12:22

To anyone already running 0.10.0

malabarba17:12:29

Give (setq cider-overlays-use-font-lock t) a try

malabarba17:12:04

We left this off-by-default in 0.10.0 to partially to reduce the surprise factor simple_smile

malabarba17:12:10

But it's worth a try

rickmoynihan17:12:39

@malabarba: it doesn't seem to make a difference for me... I think my overlays were fontlocking before setting that

malabarba17:12:27

@rickmoynihan yeah, the debugger overlays are always font locked. That variable is for regular evaluation overlays, like with C-x C-e.

malabarba17:12:26

Although now I'm not sure whether they're enabled by default. 😆

malabarba17:12:23

It's hard to remember what the defaults are.

rickmoynihan17:12:55

No worries - the setting seems to work for me simple_smile

rickmoynihan17:12:17

Loving the new cider-inspect too