towolf / vim-helm

vim syntax for helm templates (yaml + gotmpl + sprig + custom)
Other
203 stars 22 forks source link

Fix ftdetect with other protocols (like oil://) #25

Closed zoriya closed 3 months ago

zoriya commented 3 months ago

findfile does not work for other protocols (uri that contains ://). This fix simply disable Chart.yaml search if the current filepath starts with such uri.

For an example of why the current behavior is bad: Oil (netwr replacement) uses oil://path/of/directory for its file paths. Without this fix, oil's ft is always set to helm.

Fixes #24

Drowze commented 3 months ago

Thank you for the very quick patch - I'm currently using it and it fixed the issue for me :)

luisdavim commented 3 months ago

Hi, I've opened https://github.com/towolf/vim-helm/pull/26 that removes that check altogether, there was another issue related to that.