Skip to content

refactor(actions): try use pwsh/powershell/cmd/wslview to open url#294

Merged
linrongbin16 merged 1 commit intomasterfrom
refactor
Jan 19, 2026
Merged

refactor(actions): try use pwsh/powershell/cmd/wslview to open url#294
linrongbin16 merged 1 commit intomasterfrom
refactor

Conversation

@linrongbin16
Copy link
Copy Markdown
Owner

@linrongbin16 linrongbin16 commented Jan 19, 2026

Refactor #292

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.

@linrongbin16 linrongbin16 merged commit 9240e4c into master Jan 19, 2026
5 checks passed
@linrongbin16 linrongbin16 deleted the refactor branch January 19, 2026 02:05
@jtrungle
Copy link
Copy Markdown
Contributor

Hi @linrongbin16, just letting you know that this commit broke the functionality of opening from wsl - vim.fn.has("win32") and vim.fn.has("win64") returns 0 on wsl

@linrongbin16
Copy link
Copy Markdown
Owner Author

Hi @linrongbin16, just letting you know that this commit broke the functionality of opening from wsl - vim.fn.has("win32") and vim.fn.has("win64") returns 0 on wsl

hi @jtrungle , thank you for this.

I just submit a PR: #295 , could you try it and see if it works for you?

@jtrungle
Copy link
Copy Markdown
Contributor

awesome that works - thanks @linrongbin16 !

@linrongbin16
Copy link
Copy Markdown
Owner Author

awesome that works - thanks @linrongbin16 !

Thank you for testing, just merged, please pull latest commit!

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