Skip to content
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

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

Merged
merged 2 commits into from
Jun 22, 2024

Conversation

RVxLab
Copy link
Contributor

@RVxLab RVxLab commented Jun 8, 2024

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

@adalessa adalessa merged commit fb8d142 into adalessa:main Jun 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants