search

jpalardy / vim-slime

A vim plugin to give you some slime. (Emacs)
http://technotales.wordpress.com/2007/10/03/like-slime-for-vim/
MIT License
1.83k stars 223 forks source link

Remove paste_file from neovim target #435

Closed jpalardy closed 1 month ago

jpalardy commented 1 month ago

Removed dependency on g:slime_paste_file which didn't even look like it was needed 🤔

Advantages

less steps

related: https://github.com/jpalardy/vim-slime/pull/431

jpalardy commented 1 month ago

@jam1015

I very lightly tested this, as neither a neovim nor python user.

I got burned on #434 because I had misconfigured and wasn't even testing my changes 😬

As a safety net, I used chmod 000 ~/.slime_paste so that any attempt to read or write will blow up.

jam1015 commented 1 month ago

@jpalardy I just tested it and it seems to work on the neovim both for ipython (mentioned in the comment justifying the paste file) and for the vanilla python REPL, and for R. tried indented code blocks for each. ipython needs g:slime_python_ipython and bracketed paste doesn't work but I think that is part of the neovim target. ptpython, another python REPL that I like, doesn't work in the neovim target and seems to need bracketed paste. I don't know if a pastefile can help with that, but, since we've established that it isn't even used by the Neovim target, I would merge this PR.

jpalardy commented 1 month ago

thanks for confirming 🙏

we'll see what happens 😄