Helm syntaxe and file detection still not complete

[nabiltntn]

Firs, thank you for the release of the extension !

With 0.0.1 version, syntaxe highlighting is OK and reserved Helm variable are detected but some parts are still detected as YAML errors
Example :

Also, file are not automatically detected as helm files, I had to select language Helm explicitly

Installed helm_ls version :

Helm-ls version: 0.1.0
Git commit: Homebrew
Build type: Release
Build time: 2024-09-05T15:10:12Z
Golang: go1.23.1
Compiled by: Homebrew

[narqo]

Also, file are not automatically detected as helm files, I had to select language Helm explicitly

I struggled with the same and was about to remove the extension. But accidentally I've bumped into this snippet from Zed's documentation about Helm support that solved it:

// .zed/settings.json
"file_types": {
    "Helm": [
      "**/templates/**/*.tpl",
      "**/templates/**/*.yaml",
      "**/templates/**/*.yml",
      "**/helmfile.d/**/*.yaml",
      "**/helmfile.d/**/*.yml"
    ]
  }

It'd be great if it was cross-referenced from the extension's readme, as well as the need to have helm-ls installed globally (until #7 is resolved, at least).

[nabiltntn]

helm-ls installed globally (until #7 is resolved, at least).

Totally agree, had the same issue and even the error message says Helm is not installed, while it's helm-ls that was missing

I think there is currently a PR to install it automatically with extension like it's done other languages like groovy

[cabrinha]

helm-ls installed globally (until #7 is resolved, at least).

Totally agree, had the same issue and even the error message says Helm is not installed, while it's helm-ls that was missing

I think there is currently a PR to install it automatically with extension like it's done other languages like groovy

Just need the conflicts resolved and we can move that one forward.

[cabrinha]

helm-ls installed globally (until #7 is resolved, at least).

Totally agree, had the same issue and even the error message says Helm is not installed, while it's helm-ls that was missing

I think there is currently a PR to install it automatically with extension like it's done other languages like groovy

@nabiltntn I think the original author abandoned, feel free to take over.