Fork me on GitHub
#cider
<
2017-01-23
>
dpsutton03:01:19

@bozhidar what's the motivation behind cider-ensure-op-supported?

dpsutton03:01:32

i get that its to make sure the backend can handle it, but don't they go in lockstep

dpsutton03:01:43

ie, if CIDER has it, cider-nrepl has it?

dpsutton03:01:11

and this comes up because it's not connection aware. Ie, clj repl supports it but the connection you are running against doesn't

stuartrexking03:01:13

cider-find-var returns that the symbol is not resolved for all symbols. Is this the best way to move around? If so, is there some config I’m missing?

dpsutton03:01:17

the code would have no way to know

dpsutton03:01:38

@stuartrexking are you trying to navigate around code?

dpsutton03:01:53

I use M-. and M-, to push and pop respectively

dpsutton03:01:59

make sure that the code is loaded

dpsutton03:01:05

ie, load it in the repl

dpsutton03:01:12

otherwise its possible it has no knowledge of them

stuartrexking03:01:15

Let me try that.

dpsutton03:01:16

yeah its not through context

dpsutton03:01:24

it's asking the java backend what the symbol is

dpsutton03:01:27

and that has to know it

dpsutton03:01:08

hope its a seamless experience and fun!

bozhidar03:01:03

@dpsutton some people might use an incompatible cider-nrepl version or might not be using cider-nrepl at all (the most basic cider functionality still works without it). cider-ensure-op-supported exists mostly to prevent some ugly and confusing errors. It's not like people won't get some warnings when they start CIDER but I've noticed many people simply ignore them. Perhaps we can drop this or come up with some simpler scheme to verify compatibility.

bozhidar03:01:36

I remembered one more thing - you can actually enable/disable certain middleware on demand (depending on the way you're using cider-nrepl). Probably this was the main reason for adding cider-ensure-op-supported.

dpsutton03:01:14

Is it safe to assume that if a feature works on any connection it works on every connection?

bozhidar03:01:37

you can be connected to different servers with different capabilities

dpsutton03:01:57

Awesome. Exactly the into I need. Thanks

dpsutton03:01:13

I don't do any remote work with it so needed to know that

bozhidar03:01:14

e.g. one might be enabling all the middleware, one might not be, one might be using an outdated snapshot and so on

bozhidar03:01:30

it's a pity I never found time for this

richiardiandrea03:01:40

the above looks trivial to achieve, all the pieces are there for it to be configurable, probably only an explicit cider-nrepl-version which needs to be passed in 3 places (the places where the dependencies are calculated for the 3 build tools) is necessary...no promises of course but I'm already kind of looking at that part

richiardiandrea03:01:59

@bozhidar should we disable the version check if we see that cider-nrepl-version is set?

richiardiandrea04:01:38

I will actually wait for my other PR to be in so that I won't need to rebase 😓

bozhidar04:01:16

I don't understand the question. More details, please. 🙂

bozhidar04:01:33

@richiardiandrea You've made a small mistake in your commit message - change it and I'll merge the PR right away.

richiardiandrea04:01:20

@bozhidar done and also sent a README PR on cider-nrepl 😄

bozhidar04:01:49

I'll take a look at it right away

bozhidar04:01:13

guess we're finally done with this very long-running task 🙂

richiardiandrea04:01:31

I was kind of ashamed of not having a commit in cider while being such an avid fan 😄

richiardiandrea04:01:20

also probably we should suppress the wiki and/or point to the doc only, people might think "oh the wiki is empty, there is no doc in this project" 😄

richiardiandrea05:01:00

Is there support boot for the cljs repl in cider? It looks like there is only cider-cljs-lein-repl.

dpsutton05:01:30

that gets called for every clojurescript jack in

dpsutton05:01:51

it should be renamed to cider-cljs-repl-command or something

richiardiandrea05:01:09

yeah, let me try if it actually works in boot

richiardiandrea05:01:31

yep it looks like it is called for boot as well

dpsutton05:01:28

yeah that code pathway is just registered in the main cider-jack-in process. it doesn't have access to the project type and other things

dpsutton05:01:39

or at least withotu running the functions that got that info in the first place

dpsutton05:01:54

i looked at possibly letting that have the same info as the main task start up

dpsutton05:01:26

but its registered as a callback and applied to the connection buffer (i think) and it was going to be a lot of work to make a closure or something

richiardiandrea05:01:32

got it, at least we can change the name

richiardiandrea05:01:42

but it will be breaking 😄

richiardiandrea05:01:55

I'll attempt a cider-cljs-repl-form

richiardiandrea05:01:49

also for boot probably piggieback and weasel could be auto-injected

bozhidar05:01:26

I think this is just an oversight because originally this functionality relied on lein - probably we need to add a similar defcustom for boot, otherwise everyone will have to use dir-locals when switching between project types

bozhidar05:01:59

good idea about the wiki - never really liked the github wiki, so it remained empty

richiardiandrea05:01:11

mmm true about the first

bozhidar05:01:55

my talk with @dpsutton about the tooling session reminded me that it might be a good to have some glossary section in the manual explaining some potentially confusing terms

bozhidar05:01:41

wiki disabled

richiardiandrea05:01:43

today is my personal cider day 🙂

richiardiandrea05:01:57

some (small) doc PR opened

richiardiandrea05:01:16

but now it is time to rest 😉

ddellacosta17:01:46

okay I am banging my head against the wall trying to figure this out, and C-c C-o doesn’t seem to be it (?) : how do I clear the repl buffer??

ddellacosta17:01:05

I somehow lost my muscle memory this past weekend from using intero-mode, which uses a different key combo for this

ddellacosta17:01:40

I’m having trouble finding whatever it was I was using in the cider docs

ddellacosta17:01:10

I guess I’m looking for whatever cider-repl-clear-buffer is set to by default?

ddellacosta17:01:18

NEVERMIND sorry for the noise

bozhidar17:01:31

Just for the sake of completeness - it's C-u C-c C-o (or you can simply use the menu bar entry for this command)

ddellacosta19:01:53

bozhidar yeah sorry, that’s what I came up with—basically I missed the part about the prefix (`C-u`)

ddellacosta19:01:03

which is in the docs, to be clear, I just didn’t get it