Exafunction / codeium.nvim

A native neovim extension for Codeium
MIT License
644 stars 50 forks source link

remove nui dependency? #27

Closed chrisgrieser closed 1 year ago

chrisgrieser commented 1 year ago

as far as I can tell, nui is only needed for authentication, isn't it? I tried removing it and everything seems to work fine, still.

Given that codeium.nvim is the only plugin on my machine that requires nui, and nui isn't even used anymore, it seemed a bit pointless to still have nui around. And just for the one-time authentication, one shouldn't have to add an extra dependency they potentially never need anymore.

Would it maybe make sense to remove the nui dependency and use vim.notify and vim.ui.input instead?

jcdickinson commented 1 year ago

Yeah, it would make sense. I wasn't aware of these when I wrote the auth code.