sindrets / diffview.nvim

Single tabpage interface for easily cycling through diffs for all modified files for any git rev.
Other
3.74k stars 105 forks source link

healthcheck shows warnings from call to deprecated health reporting api #361

Closed gongfarmer closed 1 year ago

gongfarmer commented 1 year ago

Running :checkhealth diffview shows multiple warnings for several weeks now.

The actual output for me is reproduced below. I would expect to see this same output but without all the WARNING lines.

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

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

Checking plugin dependencies ~
- WARNING vim.health.report_ok() is deprecated, use vim.health.ok() instead. :help |deprecated|
  This feature will be removed in Nvim version 0.11
- OK nvim-web-devicons installed.

Checking VCS tools ~
- WARNING vim.health.report_info() is deprecated, use vim.health.info() instead. :help |deprecated|
  This feature will be removed in Nvim version 0.11
- The plugin requires at least one of the supported VCS tools to be valid.
- OK Git found.
- WARNING vim.health.report_warn() is deprecated, use vim.health.warn() instead. :help |deprecated|
  This feature will be removed in Nvim version 0.11
- WARNING Git version is outdated! Some functionality might not work as expected, or not at all! Current: 2.30.2, wanted: 2.31.0
- WARNING Configured `hg_cmd` is not executable: 'hg'
- WARNING vim.health.report_error() is deprecated, use vim.health.error() instead. :help |deprecated|
  This feature will be removed in Nvim version 0.11
- ERROR No valid VCS tool was found!
sindrets commented 1 year ago

The remarks about inaction are unnecessary. These functions have only been soft deprecated in nightly. They're not deprecated in any stable release, and nobody has reported it until now.