-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
vim.fn: throw error when trying to use API function #13875
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
cool, maybe a test? :) |
|
Sharing just because more people might face the same problem I'm having. This commit broke quite a few plugins I use. Based on the error:
Given that plenary and popup are base for other plugins I imagine a bigger impact. grep command$ rg 'fn.nvim_create_buf'
site/pack/packer/start/telescope.nvim/lua/telescope/builtin/internal.lua
320: -- local buf = vim.fn.nvim_create_buf(false, true)
site/pack/packer/start/plenary.nvim/scratch/window_test.lua
8: local bufnr = vim.fn.nvim_create_buf(false, true)
20: buf_text = vim.fn.nvim_create_buf(false, true)
22: buf_border = vim.fn.nvim_create_buf(false, true)
site/pack/packer/start/plenary.nvim/lua/plenary/popup.lua
37: bufnr = vim.fn.nvim_create_buf(false, true)
site/pack/packer/start/plenary.nvim/lua/plenary/window/float.lua
50: local bufnr = options.bufnr or vim.fn.nvim_create_buf(false, true)
91: local primary_bufnr = vim.fn.nvim_create_buf(false, true)
178: local bufnr = options.bufnr or vim.fn.nvim_create_buf(false, true)
site/pack/packer/start/nvim-lspconfig/lua/lspconfig/_lspui.lua
94: local bufnr = options.bufnr or vim.fn.nvim_create_buf(false, true)
site/pack/packer/start/popup.nvim/lua/popup/init.lua
38: bufnr = vim.fn.nvim_create_buf(false, true)I'm not sure how to proceed, again, just want to share what I'm experiencing. |
|
@kassio Since plugins had the freedom to be inconsistent in the usage of Thankfully, the fix for these errors is pretty trivial/straightforward, such that any broken plugins should be able to be fixed fairly quickly; at the very least, I was able to open PRs on the repositories you mentioned to fix the errors in those places (the one on |
See neovim/neovim#13875 for details
See neovim/neovim#13875. Switching over the commented out code so that we don't face troubles when uncommenting it
See neovim/neovim#13875 for details
See neovim/neovim#13875 for details.
|
:/ this is THE WORST PR ever. Seriously!!!! like wtf. I know it's the right move and I totally agree with the reasons given. But it broke everything! There should've been some type of announcement, coordination with other authors, or something before merge. NO HARD FEELINGS! |
See neovim/neovim#13875 for details.
|
We did change our routines for breaking changes after this. Now there is this issue #14090 you can subscribe to for breaking changes, replacing the old wiki page (as apparently you cannot "follow" a github wiki page). |
See neovim/neovim#13875. Switching over the commented out code so that we don't face troubles when uncommenting it
See neovim/neovim#13875. Switching over the commented out code so that we don't face troubles when uncommenting it
See neovim/neovim#13875 for details.
No description provided.