clj-kondo

teodorlu 2026-03-02T09:28:34.171449Z

Is it recommended to add .clj-kondo/inline-configs to .gitignore? https://github.com/clj-kondo/clj-kondo/blob/master/doc/config.md#inline-macro-configuration

borkdude 2026-03-02T09:36:30.746649Z

https://clojurians.slack.com/archives/CPABC1H61/p1772444066646999 Maybe that is a good reason to check them in

cjohansen 2026-03-02T09:38:36.653779Z

We're trying to figure out what to do. My stance is that checking in parts of dependencies that we already specify to a specific version in deps.edn should be unnecessary. It opens up for having library versions out of sync with their own linting rules.

borkdude 2026-03-02T09:39:30.040139Z

If they are out of sync they will be overwritten

cjohansen 2026-03-02T09:40:14.565009Z

By what? the import-configs script?

borkdude 2026-03-02T09:45:38.661569Z

by clojure-lsp or a manual import

borkdude 2026-03-02T09:46:17.958989Z

it's not an ideal situation. the ideal situation would be to load stuff directly from the classpath I guess. something I want to look into

❤️ 2
borkdude 2026-03-02T09:46:36.417509Z

but for inlined configs that would be difficult since the config is hidden in between the code

👍 1
cjohansen 2026-03-02T09:47:39.483809Z

I had expected lsp to do this, hence my question. Another option is to write a git-hook that runs the import when deps.edn changes

borkdude 2026-03-02T09:58:42.448249Z

clojure-lsp should be doing this. but maybe you first have to restart your workspace, it's not common to run init stuff on a file change I think

borkdude 2026-03-02T09:59:13.492779Z

if you're using emacs, it's lsp-workspace-restart if you're using lsp-mode

borkdude 2026-03-02T09:59:43.309529Z

what editor are you using btw @christian767, given that your colleague is heavily into emacs, I suspect emacs...?

cjohansen 2026-03-02T10:24:10.523999Z

We're all using the same Emacs setup 😄

teodorlu 2026-03-02T10:24:30.699829Z

https://github.com/magnars/emacsd-reboot/

cjohansen 2026-03-02T10:24:51.448659Z

lsp-workspace-restart works, but must be run manually

cjohansen 2026-03-02T10:25:07.778139Z

So we're looking for some way to make it happen automatically - but only when needed, since it takes some time.

borkdude 2026-03-02T10:25:54.858099Z

since you're using the same emacs config, I could imagine a way to script this in emacs. take the hash of deps.edn, when it changes, run lsp-workspace-restart?

borkdude 2026-03-02T10:26:14.298979Z

this could create problems when you're actually editing deps.edn though, so not sure about the UX

cjohansen 2026-03-02T10:54:12.294099Z

True, we could just do it in Emacs. I had originally planned to do it after pulling/committing, so that should solve the UX bit.

borkdude 2026-03-02T10:54:36.958379Z

after pulling/committing could be done with a git hook

cjohansen 2026-03-02T10:54:57.000749Z

Full circle 😄

teodorlu 2026-03-02T14:00:40.151319Z

We ended up with a hook from our (start) function. Thanks for the assistance, @borkdude! 🙏

borkdude 2026-03-02T14:01:32.216279Z

🙇