Rishikesh Vaishnav
Rishikesh Vaishnav
SGTM, if it's described as such in the readme we should probably go with that. I have no issues either way though.
(Sorry, I'm ramping up at a new job but will get back to this soon. This PR isn't exactly what I think we want yet, and I do like the...
(CC: @Julian)
Alright it should be ready for a detailed review at this point! @ [rish987/nvim-lspconfig-test](https://github.com/rish987/nvim-lspconfig-test). I've added you both as collaborators, feel free to leave comments/make any changes as you see...
Hi there, thanks for your interest! I've just recently started to get back into testing. In the experimental repo [rish987/nvim-lspconfig-test](https://github.com/rish987/nvim-lspconfig-test) I have the requirement to "formalize" the install steps of...
Hi @akinsho, thanks for the review! I'm marking this as a draft because I wanted to get your opinion on my recent idea in #289. Essentially, yes I agree that...
@akinsho I've overhauled this PR, removing strategies in the process. I added the "Smart-Toggling" section to the README explaining the new design, let me know what you think!
@akinsho Sorry for the long wait, but I'm back now. I've taken your comments into consideration and done a refactor. The main difference is that we're now toggling w.r.t. a...
Thanks for the feedback @akinsho. I've just added logic to default to the original behavior of `smart_toggle` when `get_ctx == nil` (which was to open the first-opened terminal in the...
Another smart-toggle helper we could add is one that opens the last-used terminal, which I think could be very useful to avoid having to provide a count before `` when...