Hi there! Given the two following Clojure files:

; file myprotocol.clj
(ns myprotocol)

(defprotocol MyProtocol
  (method-one [this arg]))

; file mytype.clj
(ns mytype
  (:require [myprotocol])
  (:import [myprotocol MyProtocol]))

(deftype MyType []
  MyProtocol
  (method-one [_ arg] (str "arg is:" arg)))

@pelletier When you’re implementing a protocol, you need an extra this parameter.

not sure what you mean

(method-one [this arg] ...) ?

Ah, no, sorry, you’re right

I think the problem is that you don’t need the import. What you’re doing there is implementing the interface generated by the protocol, and the method there will be called method_one.

I think you can delete the import, (:require [myprotocol :as p]) and then (deftype MyType [] p/MyProtocol ...)

ha

nice

got it

thanks 👍

No worries!

oh...that explains a lot

It would probably be a good idea to have a warning that you’re implementing a protocol interface rather than the protocol itself.

https://github.com/cursive-ide/cursive/issues/1689

The Tools->REPL menu acts weird on a lib when I try to run tests. Sometimes the action to run tests is enabled and sometimes not, sometimes I can move to the tests namespace in REPL and sometimes not. What's the surest way to make it work consistently? I don't mind adding a bunch of redundant configuration if necessary.

By moving to tests ns I mean requiring it or (in-ns ...) to it.

I've noticed in the most recent EAP's that Refresh Leiningen Projects only works when you are in a project.clj file

The old behaviour allowed it from anywhere (I think)

Is this an intentional change?

https://xkcd.com/1172/