https://github.com/clojure-emacs/inf-clojure/pull/200, and https://github.com/clojure-emacs/inf-clojure/pull/201. I would appreciate getting these merged, if you would like to help with testing (by just using it) https://github.com/JasonKDarby/.emacs.d/blob/cedb915ae2a70b11ce6adc05f22a211596f07869/init.el#L308 (or however you want).
@jasonkdarby i left a comment on the switch-to-repl PR about the way CIDER uses (buffer-list) rather than trying to keep a variable up to date
I do really like the idea of not keeping the extra variable around but I don't understand the workflow. I don't personally use it, I just cycle through other-window (with M-o, I'm not entirely insane), so I've never used the similar functionality in CIDER
the workflow is basically the same as you wanted to provide in your PR?
ie, C-c C-z takes you to the repl buffer when you're in a clojure mode buffer. If you are in a repl buffer, C-c C-z takes you to the last inf-clojure-minor-mode buffer (aka clojure source code buffer)
which is what your PR does except it uses a variable to keep track of it and requires you to use the shortcut instead of "just working"ā¢
Ok, yeah I understood it as "`C-c C-z` takes you to the repl buffer when you're in a clojure mode buffer. If you are in a repl buffer, C-c C-z takes you to the last inf-clojure-minor-mode buffer you invoked C-c C-z from."
I'll give it a shot with your suggestion
right. so just drop the requirement "you invoked C-c C-z from" since that could lead to bewildering results and requires you to remember how you got to the repl
just grab the first buffer from (buffer-list) that has inf-clojure-minor-mode active and that's the target buffer
lol can you imagine what it's like to work with me
i'm sure you're great. you are improving tooling š sounds like a great coworker
FYI I think I fixed it. To anybody interested, this fix is part of JasonKDarby/inf-clojure:master, along with my other PR.
@jasonkdarby i'll use these two changesets on tuesday and get them merged if all good
iām applying the patches now. sorry about that
@dpsutton did you get a chance to use them?
hey @dpsutton any progress on this issue? https://github.com/nrepl/nrepl/issues/272 I'm also using nrepl + tty & inf-clojure
You can put this in your configuration in the meantime https://github.com/JasonKDarby/.emacs.d/blob/cedb915ae2a70b11ce6adc05f22a211596f07869/init.el#L327