Skip to content

Commit

Permalink
Address review comments (remove obsolete option)
Browse files Browse the repository at this point in the history
  • Loading branch information
ikappaki committed May 27, 2024
1 parent 8e99af4 commit f48f0e2
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 14 deletions.
1 change: 0 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
- [#3624](https://github.com/clojure-emacs/cider/pull/3624): Support new `cider.clj-reload/reload` cider-nrepl middleware.
- adds `cider-ns-code-reload-tool` defcustom, defaulting to `'tools.namespace`.
- you can change it to `'clj-reload` to use [clj-reload](https://github.com/tonsky/clj-reload) instead of [tools.namespace](https://github.com/clojure/tools.namespace).

- [#3682](https://github.com/clojure-emacs/cider/issues/3682): Add `cider-jack-in` support for [Basilisp](https://github.com/basilisp-lang/basilisp) (Python)

### Changes
Expand Down
15 changes: 2 additions & 13 deletions cider.el
Original file line number Diff line number Diff line change
Expand Up @@ -283,13 +283,6 @@ By default we favor the project-specific shadow-cljs over the system-wide."
:safe #'stringp
:package-version '(cider . "1.14.0"))

(defcustom cider-basilisp-global-options
nil
"Command line options used to execute Basilisp."
:type 'string
:safe #'stringp
:package-version '(cider . "1.14.0"))

(defcustom cider-basilisp-parameters
"nrepl-server"
"Params passed to Basilisp to start an nREPL server via `cider-jack-in'."
Expand All @@ -304,7 +297,6 @@ By default we favor the project-specific shadow-cljs over the system-wide."
(make-obsolete-variable 'cider-gradle-global-options 'cider-gradle-parameters "1.8.0")
(make-obsolete-variable 'cider-babashka-global-options 'cider-babashka-parameters "1.8.0")
(make-obsolete-variable 'cider-nbb-global-options 'cider-nbb-parameters "1.8.0")
(make-obsolete-variable 'cider-basilip-global-options 'cider-basilisp-parameters "1.8.0")

(defcustom cider-jack-in-default
(if (executable-find "clojure") 'clojure-cli 'lein)
Expand Down Expand Up @@ -518,7 +510,7 @@ Throws an error if PROJECT-TYPE is unknown."
('shadow-cljs cider-shadow-cljs-global-options)
('gradle cider-gradle-global-options)
('nbb cider-nbb-global-options)
('basilisp cider-basilisp-global-options)
('basilisp nil)
(_ (user-error "Unsupported project type `%S'" project-type))))

(defun cider-jack-in-params (project-type)
Expand Down Expand Up @@ -995,10 +987,7 @@ middleware and dependencies."
global-opts
(unless (seq-empty-p global-opts) " ")
params))
('basilisp (concat
global-opts
(unless (seq-empty-p global-opts) " ")
params))
('basilisp params)
(_ (error "Unsupported project type `%S'" project-type))))


Expand Down
2 changes: 2 additions & 0 deletions doc/modules/ROOT/pages/platforms/basilisp.adoc
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
= Basilisp Integration with CIDER
https://github.com/basilisp-lang/basilisp[basilisp]

since CIDER 1.14

== Overview

Basilisp aims to enable writing Clojure programs on Python with full Python interoperability. It is highly compatible with Clojure.
Expand Down

0 comments on commit f48f0e2

Please sign in to comment.