refactor(lsp): use lsp server and client interfaces #191
qvalentinAnnotations
10 errors and 2 warnings
|
lint (1.21.5, ubuntu-latest):
internal/adapter/yamlls/client.go#L10
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint (1.21.5, ubuntu-latest):
internal/adapter/yamlls/client.go#L15
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint (1.21.5, ubuntu-latest):
internal/adapter/yamlls/yamlls.go#L48
Error return value of `io.Copy` is not checked (errcheck)
|
|
lint (1.21.5, ubuntu-latest):
internal/adapter/yamlls/yamlls.go#L75
ineffectual assignment to ctx (ineffassign)
|
|
lint (1.21.5, ubuntu-latest):
internal/handler/completion.go#L91
string `Chart` has 3 occurrences, make it a constant (goconst)
|
|
lint (1.21.5, ubuntu-latest):
internal/handler/completion.go#L93
string `Values` has 3 occurrences, make it a constant (goconst)
|
|
lint (1.21.5, ubuntu-latest):
internal/handler/text_document.go#L36
Error return value of `h.client.PublishDiagnostics` is not checked (errcheck)
|
|
lint (1.21.5, ubuntu-latest):
internal/handler/text_document.go#L56
ineffectual assignment to err (ineffassign)
|
|
lint (1.21.5, ubuntu-latest):
internal/handler/text_document.go#L58
Error return value of `h.client.PublishDiagnostics` is not checked (errcheck)
|
|
lint (1.21.5, ubuntu-latest):
internal/lsp/lint.go#L26
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint (1.21.5, ubuntu-latest)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-go@v4. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
lint (1.21.5, ubuntu-latest)
Restore cache failed: Dependencies file is not found in /home/runner/work/helm-ls/helm-ls. Supported file pattern: go.sum
|