Skip to content

Feat: yamlls

Feat: yamlls #56

Triggered via pull request November 26, 2023 18:14
Status Success
Total duration 1m 43s
Artifacts

lint.yml

on: pull_request
Matrix: lint
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 1 warning
lint (1.19.1, ubuntu-latest): internal/adapter/yamlls/diagnostics.go#L28
Error return value of `clientConn.Notify` is not checked (errcheck)
lint (1.19.1, ubuntu-latest): internal/adapter/yamlls/documentSync.go#L33
Error return value of `(go.lsp.dev/jsonrpc2.Conn).Notify` is not checked (errcheck)
lint (1.19.1, ubuntu-latest): internal/adapter/yamlls/documentSync.go#L42
Error return value of `(go.lsp.dev/jsonrpc2.Conn).Notify` is not checked (errcheck)
lint (1.19.1, ubuntu-latest): internal/adapter/yamlls/documentSync.go#L73
Error return value of `(go.lsp.dev/jsonrpc2.Conn).Notify` is not checked (errcheck)
lint (1.19.1, ubuntu-latest): internal/adapter/yamlls/initization.go#L11
var-naming: method parameter workspaceUri should be workspaceURI (revive)
lint (1.19.1, ubuntu-latest): internal/adapter/yamlls/initization.go#L25
Error return value of `(go.lsp.dev/jsonrpc2.Conn).Call` is not checked (errcheck)
lint (1.19.1, ubuntu-latest): internal/adapter/yamlls/trimTemplate.go#L76
var-naming: don't use underscores in Go names; var function_name should be functionName (revive)
lint (1.19.1, ubuntu-latest): internal/adapter/yamlls/yamlls.go#L15
exported: type name will be used as yamlls.YamllsConnector by other packages, and that stutters; consider calling this Connector (revive)
lint (1.19.1, ubuntu-latest): internal/handler/completion.go#L82
string `Chart` has 3 occurrences, make it a constant (goconst)
lint (1.19.1, ubuntu-latest): internal/handler/completion.go#L84
string `Values` has 3 occurrences, make it a constant (goconst)
lint (1.19.1, ubuntu-latest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-go@v2, actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/