ayamir / nvimdots

A well configured and structured Neovim.
BSD 3-Clause "New" or "Revised" License
2.82k stars 450 forks source link

replace `vim.tbl_islist()` with `vim.islist()` after `NVIM v0.10.0-dev-2560` #1245

Closed CharlesChiuGit closed 2 days ago

CharlesChiuGit commented 2 months ago

https://github.com/akinsho/bufferline.nvim/commit/a6ad228f77c276a4324924a6899cbfad70541547 https://neovim.io/doc/user/news.html#news-breaking

for examples: https://github.com/ayamir/nvimdots/blob/d60116fe5edecb373a77e050a92d33884a38b4dd/lua/modules/utils/init.lua#L280-L282

CharlesChiuGit commented 2 months ago

todo

Jint-lzxy commented 2 months ago

I think we could consider postponing this change until the API stabilizes - afaik this and similar APIs have undergone numerous iterations since the initial change, and the synopsis has also experienced incompatible changes (which would render it incompatible even with older nightly builds). Plus since it's only deprecated, imo it's safe to continue using it until v0.10 is released.

CharlesChiuGit commented 2 months ago

I think we could consider postponing this change until the API stabilizes

agree~