astrocommunity icon indicating copy to clipboard operation
astrocommunity copied to clipboard

Add elin to the clojure extra pack

Open aliresool621 opened this issue 9 months ago • 2 comments

Is your feature related to a problem?

No response

Describe the new feature

https://github.com/liquidz/elin/tree/main?tab=readme-ov-file

It wont conflict with the clojure lsp, but will work with it

Additional context

No response

aliresool621 avatar Mar 07 '25 12:03 aliresool621

Feel free to add it yourself in a PR. :)

Uzaaft avatar Mar 07 '25 14:03 Uzaaft

liquidz/elin should be added as a code-runner rather than directly to the Clojure pack.

The Clojure pack already has a code runner, Conjure, which can use Clojure CLI or Babashka as the Clojure run time. Conjure is a very mature tool that has been stable for several years.

There would be conflict with Conjure and Elin if they were both in the Clojure pack. Conjure will automatically connect to a running repl or start a babashka repl when opening a Clojure file in Neovim. Adding Elin would create a duplication of resources by run a second code-runner and REPL process.

Adding elin as a code-runner would allow a user config to set conjure plugin to false and enable elin without changing the clojure pack. This would be a good first step whilst the elin tool develops. Elin is considered alpha state by the project maintainer.

practicalli-johnny avatar Apr 20 '25 21:04 practicalli-johnny