slamhound icon indicating copy to clipboard operation
slamhound copied to clipboard

support newly nrepl

Open eunmin opened this issue 10 years ago • 4 comments

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?

eunmin avatar Nov 26 '14 16:11 eunmin

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!

upgradingdave avatar Dec 18 '14 00:12 upgradingdave

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

zentrope avatar Mar 18 '15 22:03 zentrope

:+1: this pull request!

thirteen37 avatar Apr 24 '15 15:04 thirteen37

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.)

brunchboy avatar Apr 30 '15 20:04 brunchboy