stevearc / aerial.nvim

Neovim plugin for a code outline window
MIT License
1.55k stars 76 forks source link

deprecation: `vim.lsp.get_active_clients()` #375

Closed Zeioth closed 3 weeks ago

Zeioth commented 1 month ago

Did you check existing requests?

Describe the feature

The command :checkhealth vim.deprecated prints

- WARNING vim.lsp.get_active_clients() is deprecated. Feature will be removed in Nvim 0.12
  - ADVICE:
    - use vim.lsp.get_clients() instead.
    - stack traceback:
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/lsp/util.lua:18
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/lsp/init.lua:106
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/init.lua:40
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/init.lua:73
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/init.lua:147
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/init.lua:251
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/autocommands.lua:88
        vim/_editor.lua:0
    - stack traceback:
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/lsp/util.lua:18
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/lsp/init.lua:106
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/init.lua:40
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/init.lua:121
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/init.lua:149
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/init.lua:251
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/autocommands.lua:88
        vim/_editor.lua:0
    - stack traceback:
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/lsp/util.lua:18
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/lsp/init.lua:106
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/init.lua:40
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/init.lua:73
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/init.lua:248
        vim/_editor.lua:0
    - stack traceback:
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/lsp/util.lua:18
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/lsp/init.lua:40
        /home/zeioth/.local/share/nvim/lazy/aerial.nvim/lua/aerial/backends/init.lua:129
        vim/_editor.lua:0

The plugin works correctly on nvim 0.11, so all good, but it's good to keep it in mind. See: https://neovim.io/doc/user/deprecated.html

Provide background

No response

What is the significance of this feature?

strongly desired

Additional details

No response

stevearc commented 3 weeks ago

This is already using the correct method if you update to the most recent master https://github.com/stevearc/aerial.nvim/blob/75de06f8edbd0006997a19b760045753d4f6693c/lua/aerial/backends/lsp/util.lua#L20-L29