Call all clojure functions with full namespace

tpope / vim-fireplace

fireplace.vim: Clojure REPL support

https://www.vim.org/scripts/script.php?script_id=4978

1.75k stars 139 forks source link

Call all clojure functions with full namespace #294

Closed axel-angel closed 5 years ago

axel-angel commented 7 years ago

Hello,

I use this plugin everyday at my work and one of my teammate found a way to broke by using:

(ns thingy.core
  (:refer-clojure :exclude [resolve]) …

So I decided to patch and do this PR to fix this issue. What do you think?

tpope commented 7 years ago

Not saying no, but is there a reason you're not running CIDER? I'm less strict about namespacing in the no CIDER fallback code.

axel-angel commented 7 years ago

I'm not really sure I do, I put the lein dependency and then tried to add cider.vim but this last doesn't seem to be loaded at all. I don't see any difference.

set runtimepath^=~/.vim/bundle/vim-cider/plugin/cider.vim doesn't seem to help

EDIT: Actually cider.vim is loaded but I still don't see any difference

tpope commented 7 years ago

Not cider.vim, this is the first I've heard of that. You need https://github.com/clojure-emacs/cider-nrepl in your ~/.lein/profiles.clj.