hover.hover() position is off by 1

[flrgh]

Hiya 👋

Today after updating the plugin to bb3dc1a I noticed that require("hover").hover() yields a result for $current_line + 1 instead of $current_line:

Here is my lazy.nvim package def and config for hover.nvim:

{
  "lewis6991/hover.nvim",
  event = "VeryLazy",
  config = function()
    require("hover").setup({
      init = function()
        require("hover.providers.lsp")
        require("hover.providers.man")
        require("hover.providers.gh")
      end,
      preview_opts = {
        border = nil,
      },
      title = true,
    })
  end,
}

...and I have a normal mode keymap for K which just calls require("hover").hover(). I can also reproduce the problem by running this without the keymap (e.g. :lua require("hover").hover()).

All of my other LSP mappings (definition, declaration, code-action, etc all work normally) are working fine.

Reverting to 24369e8 fixes the problem, so I suspect that there might be some unintentional breakage from these two recent commits

I'll try to writeup a minimal repro config when time permits. If there's anything else I can do to help troubleshoot in the meantime, lemme know!

[lewis6991]

No repro necessary. Should be easy to fix.

[AThePeanut4]

My PR #44 fixes this.

[lewis6991]

Closed in 3ff5bbb