nvim-telescope / telescope.nvim

Find, Filter, Preview, Pick. All lua, all the time.
MIT License
15.43k stars 824 forks source link

Have any plan to set minimal requirements to 0.10.x? #3277

Open delphinus opened 1 week ago

delphinus commented 1 week ago

Is your feature request related to a problem? Please describe. Now telescope.nvim needs Neovim 0.9.x and it is written in README. Should we think about changing this to 0.10.x?

With using 0.10.x, we can use new features, such as vim.iter, vim.system, ……, and can remove polyfills for avoiding deprecation messages (for vim.tbl_flatten and so on).

Describe the solution you'd like Update README to write this plugin needs 0.10.x.

Describe alternatives you've considered

Additional context telescope-frecency.nvim follows telescope.nvim for minimum requirements. If telescope.nvim makes minimum to 0.10.x, it is happy that we can use vim.iter in telescope-frecency.nvim.

ref https://github.com/nvim-telescope/telescope-frecency.nvim/issues/259

Conni2461 commented 1 week ago

I have a wip branch for 0.10.x, probably do it this week. After that we also most likely release 0.2

delphinus commented 1 week ago

It's nice! OK, I will set min: 0.10.x to telescope-frecency.nvim after releasing telescope.nvim 0.2. 👍🏻

clason commented 1 week ago

And I strongly encourage followup PRs to move to Neovim API as much as possible (vim.fs, vim.system, etc.; there's also a minimal async module used in several plugins that we're hoping to get into core one of these days as well as a minimal log module).

Also, nvim_strwidth is a thing and should be fast-safe, as is charwidth() (or, better yet, vim.utf_str*). So no reason for FFI tomfoolery with strings!