cider

A channel dedicated to the Clojure Interactive Development Environment that Rocks (aka CIDER). :cider:
bozhidar 2021-02-01T09:20:14.127600Z

@dpsutton Feel free to add such tips for shadow-cljs to the docs.

bozhidar 2021-02-01T09:20:25.128Z

I lot a lot of people would appreciate them.

➕ 1
tvaughan 2021-02-01T13:53:54.129500Z

With the recent changes to cider-clojure-cli-parameters and cider-clojure-cli-global-options, how can I pass options like --bind and --port to the clojure cli jack-in command?

bozhidar 2021-02-01T17:02:32.131500Z

@tvaughan Good question. 😄 I guess cider-clojure-cli-parameters was the natural place for those.

dpsutton 2021-02-01T17:03:25.132200Z

i doubt there's space for that. those need to be at the end as additional params to the main and i don't think they will work at the moment

tvaughan 2021-02-01T17:03:39.132500Z

I am/was doing (cider-clojure-cli-parameters . "-A:toolbox:koacha:kibit:user -M -m nrepl.cmdline --bind 0.0.0.0 --port 5309 --middleware '%s'") in .dir-locals.el

bozhidar 2021-02-01T17:07:40.133400Z

I guess now that we separated the aliases we can have a separate var for the params to the actual nREPL CLI. I don't know.

bozhidar 2021-02-01T17:07:58.133800Z

It's hard for me to give advice given that I don't really use tools.deps.

dpsutton 2021-02-01T17:10:31.135Z

i can fix this tonight. easiest workaround at the moment is to use a prefix when jacking in and add the --bind 0.0.0.0 --port 5309 to the end of the jack in string

tvaughan 2021-02-01T17:14:14.137300Z

> i can fix this tonight Please don't go to extra trouble on my account. I'm using straight.el and just happened to run pull-all today. I've since rolled back. I'm good right now. I'd be happy to create an issue on github, if that would be helpful

dpsutton 2021-02-01T17:17:02.137900Z

well it needs to be fixed and should have been contemplated in the original change. i was wondering why the middleware was a templated string and now i know 🙂

👍 1
bozhidar 2021-02-01T17:40:01.138700Z

This had totally slipped my mind as well. I never change the defaults, so I didn't think about the CLI options at all. I guess we need more tests. 🙂

dpsutton 2021-02-01T17:40:48.139200Z

i'll add them tonight. added an issue for this. and the tests should describe all of the pathways through it