Skip to content

feat(filetype): detect ghostty configuration files as ft=ghostty#20002

Closed
bezhermoso wants to merge 3 commits intovim:masterfrom
bezhermoso:ftype_ghostty
Closed

feat(filetype): detect ghostty configuration files as ft=ghostty#20002
bezhermoso wants to merge 3 commits intovim:masterfrom
bezhermoso:ftype_ghostty

Conversation

@bezhermoso
Copy link
Copy Markdown
Contributor

@bezhermoso bezhermoso commented Apr 18, 2026

👋

Adding file type detection support for Ghostty configuration files.

References:

Signed-off-by: Bez Hermoso <me@bez.dev>
Comment thread runtime/filetype.vim Outdated
Comment thread src/testdir/test_filetype.vim Outdated
@bezhermoso bezhermoso requested a review from WieeRd April 18, 2026 08:39
Copy link
Copy Markdown

@WieeRd WieeRd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Copy Markdown
Member

@chrisbra chrisbra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s:StarSetf() is only needed for patterns ending with a star

Comment thread runtime/filetype.vim Outdated
Comment thread runtime/filetype.vim Outdated
Co-authored-by: Christian Brabandt <cb@256bit.org>
@bezhermoso bezhermoso requested a review from chrisbra April 20, 2026 04:06
@chrisbra
Copy link
Copy Markdown
Member

thanks

@chrisbra chrisbra closed this in b30803b Apr 20, 2026
zeertzjq added a commit to neovim/neovim that referenced this pull request Apr 21, 2026
…#39267)

Problem:  filetype: ghostty config files are not recognized
Solution: Detect ghostty configuration files as ghostty filetype,
          include a simple ghostty filetype plugin (Bez Hermoso)

closes: vim/vim#20002

vim/vim@b30803b

Co-authored-by: Bez Hermoso <bezalelhermoso@gmail.com>
Co-authored-by: Christian Brabandt <cb@256bit.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants