Ihor Radchenko <yanta...@posteo.net> writes:
> Now, the docstring appears to be a bit confusing: > > (defun ob-clojure-eval-with-cider (expanded _params &optional cljs-p) > "Evaluate EXPANDED code block with PARAMS using cider. > When CLJS-P is non-nil, use a cljs connection instead of clj." > > It would be useful to mention that PARAMS argument is unused. Should I go with your initial suggestion and just replace it with _? Like (defun ob-clojure-eval-with-cider (expanded _ &optional cljs-p) "Evaluate EXPANDED code block using cider. When CLJS-P is non-nil, use a cljs connection instead of clj." But then someone will maybe wonder why there is unused argument? Or rather something like: (defun ob-clojure-eval-with-cider (expanded _params &optional cljs-p) "Evaluate EXPANDED code block using cider. When CLJS-P is non-nil, use a cljs connection instead of clj. The PARAMS from Babel are not used in this function." Writing good docstrings is hard :D Cheers, Daniel
