Skip to content
This repository has been archived by the owner on Oct 2, 2020. It is now read-only.

support newly nrepl #84

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

support newly nrepl #84

wants to merge 1 commit into from

Conversation

eunmin
Copy link

@eunmin eunmin commented Nov 26, 2014

My nREPL is cider 0.8.1.
CIDER 0.8.1 (package: 20141124.2036) (Java 1.7.0_71, Clojure 1.6.0, nREPL 0.2.6)
In that case, return value of nrepl-send-string-sync function is looks like,

(dict "err" "..." "id" "9" "session" "b614a748-c33d-49d8-9b99-4dcc678fdcf5" "out" "(ns clj-tms.http                                                                                                             
  (:require [cheshire.core :as json]                                                                                                                                                                            
            [clj-http.client :as http]))" "ns" "user" "value" "nil" "status" ("done")) 

Key :stdout change to out string.
So I fixed it.
How about?

@upgradingdave
Copy link

I'm also running cider 0.8.1 and was seeing Wrong type argument: stringp, nil. I applied this patch and now it works, thanks!

@zentrope
Copy link

I encountered the same issue with Cider 0.8.2, and this fix worked for that as well.

@thirteen37
Copy link

👍 this pull request!

@brunchboy
Copy link

I had the same issue, M-x slamhound would not work for me until I installed the patched version of slamhound.el. (Also in Cider 0.8.2.)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants