clean up

internal/adapter/yamlls/completion.go

@@ -6,7 +6,7 @@ import (
 	lsp "go.lsp.dev/protocol"
 )
 
-func (yamllsConnector *Connector) CallCompletion(ctx context.Context, params *lsp.CompletionParams) (*lsp.CompletionList, error) {
+func (yamllsConnector Connector) CallCompletion(ctx context.Context, params *lsp.CompletionParams) (*lsp.CompletionList, error) {
 	if yamllsConnector.server == nil {
 		return &lsp.CompletionList{}, nil
 	}

internal/adapter/yamlls/hover_integration_test.go

@@ -73,7 +73,7 @@ func TestYamllsHoverIntegration(t *testing.T) {
 					},
 				}, tt.word)
 				return err == nil && strings.Contains(result.Contents.Value, tt.expected)
-			}, time.Second*60, time.Second*5)
+			}, time.Second*10, time.Second*2)
 		})
 	}
 }

internal/charts/chart.go

@@ -60,8 +60,9 @@ func (c *Chart) ResolveValueFiles(query []string, chartStore *ChartStore) []*Que
 }
 
 func (c *Chart) GetValueLocation(templateContext []string) (lsp.Location, error) {
-	modifyedVar := make([]string, 0)
-	// for Charts, we make the first letter lowercase
+	modifyedVar := make([]string, len(templateContext))
+	// make the first letter lowercase since in the template the first letter is
+	// capitalized, but it is not in the Chart.yaml file
 	for _, value := range templateContext {
 		restOfString := ""
 		if (len(value)) > 1 {

internal/handler/generic_document_usecase.go

@@ -27,11 +27,10 @@ func (h *langHandler) NewGenericDocumentUseCase(
 	}
 
 	var (
-		parentNodeType string
 		nodeType       = node.Type()
 		parentNode     = node.Parent()
+		parentNodeType string
 	)
-
 	if parentNode != nil {
 		parentNodeType = parentNode.Type()
 	}