Message on startup about gh.nvim HOT 6 CLOSED

Its touched on here: https://who.ldelossa.is/posts/gh-nvim/#cli-dependencies

The background refresh keeps gh.nvim in sync with the GitHub api. But it wont do much if a pr is not open, or issue buffers arent open. Its required to get new comments / new PR commits / etc... automatically.

from gh.nvim.

Ok, thanks! Still, I think the echo on neovim startup is unnecessary. Would you mind to make it optional?

from gh.nvim.

Sounds find to me. Want to send a PR for this? Ill approve.

from gh.nvim.

I'm looking into it. I'm wondering: do we really need to start the refresh timer within init.lua:

I mean, it is understandable that we want this timer when we open a PR or issue, but do we want it already when a user does the setup() function?

My intuition would be that we could just remove that line. It would resolve my issue. But perhaps I'm missing something?

from gh.nvim.

The timer is used to refresh arbitrary Issue buffers which can be open at any time, and unrelated to having an open PR.

Its a lot simpler to just launch the timer which runs a no-op function if no Issues or PR is open, rather then trying to ref count open Issue buffers or open PRs. The no-op timer is not going to effect UX or performance when no Issue or PR buffers are open.

from gh.nvim.

Closing this as we can continue the discussion in #42.

from gh.nvim.