Skip to content

Commit 3bcc582

Browse files
committed
Deprecate the legacy connection-named aliases
`cider-current-connection', `cider-connections', `cider-map-connections' and `cider-connection-type-for-buffer' are leftovers from before the connection->REPL terminology change and have no remaining callers in CIDER. Mark them obsolete in favor of the `cider-repl' names so the byte-compiler steers any external code off them.
1 parent 436c8e3 commit 3bcc582

2 files changed

Lines changed: 8 additions & 2 deletions

File tree

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@
4242

4343
### Changes
4444

45+
- [#4033](https://github.com/clojure-emacs/cider/pull/4033): Deprecate the legacy connection-named aliases `cider-current-connection`, `cider-connections`, `cider-map-connections` and `cider-connection-type-for-buffer` in favor of their `cider-repl` counterparts.
4546
- [#4029](https://github.com/clojure-emacs/cider/pull/4029): Finish migrating to the keyword-argument request APIs and deprecate the positional shims: `cider-nrepl-send-sync-request` -> `cider-nrepl-sync-request`, `cider-nrepl-request:eval` -> `cider-nrepl-send-eval-request`, `nrepl-send-sync-request` -> `nrepl-sync-request`, `nrepl-request:eval` -> `nrepl-send-eval-request`.
4647
- [#4028](https://github.com/clojure-emacs/cider/pull/4028): Deprecate `cider-ensure-op-supported`; op support is now enforced automatically by the nREPL senders, so commands no longer need an explicit check.
4748
- [#4026](https://github.com/clojure-emacs/cider/pull/4026): Bump the injected `cider-nrepl` to 0.61.0-alpha1, which provides ClojureScript test support, the `clojure-only` op signal and the ClojureScript macroexpansion fix.

lisp/cider-session.el

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -687,12 +687,17 @@ session."
687687
(repls (cider-repls type ensure required-ops)))
688688
(mapcar function repls))))
689689

690-
;; REPLs double as connections in CIDER, so it's useful to be able to refer to
691-
;; them as connections in certain contexts.
690+
;; Legacy aliases from before the connection->REPL terminology change. They have
691+
;; no remaining users in CIDER itself; keep them working but steer code to the
692+
;; `cider-repl' names.
692693
(defalias 'cider-current-connection #'cider-current-repl)
693694
(defalias 'cider-connections #'cider-repls)
694695
(defalias 'cider-map-connections #'cider-map-repls)
695696
(defalias 'cider-connection-type-for-buffer #'cider-repl-type-for-buffer)
697+
(make-obsolete 'cider-current-connection #'cider-current-repl "1.23.0")
698+
(make-obsolete 'cider-connections #'cider-repls "1.23.0")
699+
(make-obsolete 'cider-map-connections #'cider-map-repls "1.23.0")
700+
(make-obsolete 'cider-connection-type-for-buffer #'cider-repl-type-for-buffer "1.23.0")
696701

697702

698703
(provide 'cider-session)

0 commit comments

Comments
 (0)