refactor(lsp): use lsp server and client interfaces · mrjosh/helm-ls@2f1443a

Triggered via pull request March 10, 2024 18:20

qvalentin

synchronize #65

Status Success

Total duration 1m 55s

Artifacts –

lint.yml

on: pull_request

Matrix: lint

10 errors and 2 warnings

unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)

unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)

ineffectual assignment to ctx (ineffassign)

string `Chart` has 3 occurrences, make it a constant (goconst)

string `Values` has 3 occurrences, make it a constant (goconst)

Error return value of `h.yamllsConnector.CallInitialize` is not checked (errcheck)

unused-parameter: parameter 'params' seems to be unused, consider removing or renaming it as _ (revive)

Error return value of `h.yamllsConnector.CallInitialize` is not checked (errcheck)

Error return value of `h.yamllsConnector.DidChangeConfiguration` is not checked (errcheck)

Error return value of `h.client.PublishDiagnostics` is not checked (errcheck)

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/.

Restore cache failed: Dependencies file is not found in /home/runner/work/helm-ls/helm-ls. Supported file pattern: go.sum