Skip to content

feat: respect vim.ui.open#303

Merged
linrongbin16 merged 2 commits into
linrongbin16:masterfrom
jiz4oh:master
Jun 7, 2026
Merged

feat: respect vim.ui.open#303
linrongbin16 merged 2 commits into
linrongbin16:masterfrom
jiz4oh:master

Conversation

@jiz4oh

@jiz4oh jiz4oh commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

neovim 0.10.0 had introduced a built-in way to open a file in a browser. It's better to use it than maintain the implementation in the plugin

Test Platforms

  • windows
  • macOS
  • linux

Test Hosts

Test Functions

  • Use GitLink(!) to copy git link (or open in browser).
  • Use GitLink(!) blame to copy the /blame link (or open in browser).
  • Use GitLink(!) default_branch to open the /main//master link in browser (or open in browser).
  • Use GitLink(!) current_branch to open the current branch link in browser (or open in browser).
  • Copy git link in a symlink directory of git repo.
  • Copy git link in an un-pushed git branch, and receive an expected error.
  • Copy git link in a pushed git branch but edited file, and receive a warning says the git link could be wrong.
  • Copy git link with 'file' and 'rev' parameters.

Comment thread lua/gitlinker/actions.lua Outdated
@linrongbin16 linrongbin16 merged commit bfb1bae into linrongbin16:master Jun 7, 2026
5 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