Skip to content

fix: detect helix editor by its binary name 'hx'#175

Merged
RivoLink merged 1 commit into
RivoLink:mainfrom
posteego:fix/helix-binary-name
Jun 11, 2026
Merged

fix: detect helix editor by its binary name 'hx'#175
RivoLink merged 1 commit into
RivoLink:mainfrom
posteego:fix/helix-binary-name

Conversation

@posteego

Copy link
Copy Markdown
Contributor

Helix installs as hx on PATH, not helix. This fixes auto-detection so scan_available_editors() can actually find it when installed. Also updates shell completions and tests to use the correct binary name.

@RivoLink

Copy link
Copy Markdown
Owner

Hi @posteego
Thanks for this PR, I really appreciate it.

Just one request: could you sign your commit? GPG signing is required on main branch.
If you can't sign it, no worries, I'll handle it and preserve you as the author.

GPG signing docs : https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

@posteego posteego force-pushed the fix/helix-binary-name branch from e902e45 to 16e36a1 Compare June 10, 2026 23:52
@posteego

Copy link
Copy Markdown
Contributor Author

The commit should be signed now, thanks!

@RivoLink

Copy link
Copy Markdown
Owner

Thanks you.
I am so sorry. Could you make the commit message shorter, for example:
fix: detect Helix editor by 'hx'

@posteego posteego force-pushed the fix/helix-binary-name branch from 16e36a1 to c47dfe0 Compare June 11, 2026 15:39
@posteego

Copy link
Copy Markdown
Contributor Author

No problem, commit message amended

@RivoLink

Copy link
Copy Markdown
Owner

Thank you, I really appreciate it.

@RivoLink RivoLink merged commit 3372e59 into RivoLink:main Jun 11, 2026
1 check 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