inf-clojure

Jason 2022-05-28T22:40:28.667719Z

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

dpsutton 2022-05-28T23:33:39.035909Z

@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

Jason 2022-05-29T00:06:17.012649Z

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

dpsutton 2022-05-29T00:07:11.899319Z

the workflow is basically the same as you wanted to provide in your PR?

dpsutton 2022-05-29T00:08:10.861509Z

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)

dpsutton 2022-05-29T00:08:32.414849Z

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"ā„¢

Jason 2022-05-29T00:09:41.998409Z

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

Jason 2022-05-29T00:09:50.217269Z

I'll give it a shot with your suggestion

dpsutton 2022-05-29T00:10:27.131139Z

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

dpsutton 2022-05-29T00:10:48.459639Z

just grab the first buffer from (buffer-list) that has inf-clojure-minor-mode active and that's the target buffer

šŸ‘ 1
Jason 2022-05-29T00:12:37.282059Z

lol can you imagine what it's like to work with me

dpsutton 2022-05-29T00:13:05.793219Z

i'm sure you're great. you are improving tooling šŸ™‚ sounds like a great coworker

1
Jason 2022-05-29T03:38:50.287739Z

FYI I think I fixed it. To anybody interested, this fix is part of JasonKDarby/inf-clojure:master, along with my other PR.

dpsutton 2022-05-28T22:49:03.062419Z

@jasonkdarby i'll use these two changesets on tuesday and get them merged if all good

2
dpsutton 2022-06-03T15:08:54.120909Z

i’m applying the patches now. sorry about that

Jason 2022-06-02T17:02:01.052359Z

@dpsutton did you get a chance to use them?

wanz 2022-05-28T04:30:33.207559Z

hey @dpsutton any progress on this issue? https://github.com/nrepl/nrepl/issues/272 I'm also using nrepl + tty & inf-clojure

Jason 2022-05-28T22:14:19.478909Z

You can put this in your configuration in the meantime https://github.com/JasonKDarby/.emacs.d/blob/cedb915ae2a70b11ce6adc05f22a211596f07869/init.el#L327

šŸ‘ 1