-
-
Notifications
You must be signed in to change notification settings - Fork 104
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support clj-reload as well as tools.namespace reload since it's now built into nREPL #574
Labels
Comments
dharrigan
added a commit
to dharrigan/conjure
that referenced
this issue
May 6, 2024
Since version 0.46.0, nREPL has supported the `clj-reload` library in addition to the standard `tools.namespace` library for facilitating reloading namespaces. This commit introduces a new Clojure nREPL variable to allow switching to the new `clj-reload` library. The variable is: `g:conjure#client#clojure#nrepl#refresh#backend` The default is to continue to use `tools.namespace`, but if it is set to `clj-reload`, i.e., `g:conjure#client#clojure#nrepl#refresh#backend = 'clj-reload` It will then use the new backend. fixes Olical#574 -=david=-
This was referenced May 6, 2024
dharrigan
added a commit
to dharrigan/conjure
that referenced
this issue
May 6, 2024
Since version 0.46.0, nREPL has supported the `clj-reload` library in addition to the standard `tools.namespace` library for facilitating reloading namespaces. This commit introduces a new Clojure nREPL variable to allow switching to the new `clj-reload` library. The variable is: `g:conjure#client#clojure#nrepl#refresh#backend` The default is to continue to use `tools.namespace`, but if it is set to `clj-reload`, i.e., `g:conjure#client#clojure#nrepl#refresh#backend = 'clj-reload'` It will then use the new backend. fixes Olical#574 -=david=-
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As requested by @dharrigan, we should support both methods. We should add an enum option that lets you swap to the new method easily. It should use the same mappings to control the other system.
https://github.com/tonsky/clj-reload
Original Discord message:
The text was updated successfully, but these errors were encountered: