adalessa / laravel.nvim

Plugin for Nvim to work with laravel projects.
MIT License
235 stars 13 forks source link

Make use of Nvim's new vim.health functions if available #91

Closed RVxLab closed 3 weeks ago

RVxLab commented 1 month ago

The vim.health.report_ functions have been deprecated as of Nvim 0.10 and will be removed in 0.11.

This PR adds aliases to each of the health functions that are used to ensure the new functions are used when possible and maintaining backwards compatibility for 0.9 and before.

Tested locally on my Neovim 0.10 instance on MacOS:

==============================================================================
laravel: require("laravel.health").check()

Laravel ~

External Dependencies ~
- OK rg installed

Plugin Dependencies ~
- OK Null LS is installed
- OK luasnip is installed
- OK Nui is installed
- OK Telescope is installed

Environment ~
- OK Environment setup successful
- Name: local
- Condition: 
- {
    executable = { "php" }
  }
- Commands: 
- {}

Composer Dependencies ~
- OK Composer dependency `laravel/tinker` is installed

Using v2.1.1:

laravel: require("laravel.health").check()

- WARNING vim.health.report_start() is deprecated, use vim.health.start() instead. :help |deprecated|
  Feature will be removed in Nvim 0.11

Laravel ~

External Dependencies ~
- WARNING vim.health.report_ok() is deprecated, use vim.health.ok() instead. :help |deprecated|
  Feature will be removed in Nvim 0.11
- OK rg installed

Plugin Dependencies ~
- OK Null LS is installed
- OK luasnip is installed
- OK Nui is installed
- OK Telescope is installed

Environment ~
- OK Environment setup successful
- WARNING vim.health.report_info() is deprecated, use vim.health.info() instead. :help |deprecated|
  Feature will be removed in Nvim 0.11
- Name: local
- Condition: 
- {
    executable = { "php" }
  }
- Commands: 
- {}

Composer Dependencies ~
- OK Composer dependency `laravel/tinker` is installed