Feat: yamlls #35
Annotations
10 errors and 1 warning
lint (1.19.1, ubuntu-latest):
internal/adapter/yamlls/diagnostics.go#L26
Error return value of `clientConn.Notify` is not checked (errcheck)
|
lint (1.19.1, ubuntu-latest):
internal/adapter/yamlls/documentSync.go#L19
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#L28
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#L59
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#L19
Error return value of `(go.lsp.dev/jsonrpc2.Conn).Call` is not checked (errcheck)
|
lint (1.19.1, ubuntu-latest):
internal/adapter/yamlls/initization.go#L27
exported: type name will be used as yamlls.YamllsSettings by other packages, and that stutters; consider calling this Settings (revive)
|
lint (1.19.1, ubuntu-latest):
internal/adapter/yamlls/trimTemplate.go#L27
superfluous-else: if block ends with a break statement, so drop this else and outdent its block (revive)
|
lint (1.19.1, ubuntu-latest):
internal/adapter/yamlls/trimTemplate.go#L31
var-naming: don't use underscores in Go names; var if_action_condition should be ifActionCondition (revive)
|
lint (1.19.1, ubuntu-latest):
internal/adapter/yamlls/yamlls.go#L14
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/adapter/yamlls/yamlls.go#L19
unused-parameter: parameter 'workingDir' seems to be unused, consider removing or renaming it as _ (revive)
|
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/
|