Merge pull request #100 from mrjosh/fix/allBuiltInObjectsForUsecase #383
Annotations
7 errors and 2 warnings
|
internal/adapter/yamlls/client.go#L10
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
internal/adapter/yamlls/client.go#L15
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
internal/adapter/yamlls/client.go#L20
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
internal/adapter/yamlls/yamlls.go#L68
Error return value of `io.Copy` is not checked (errcheck)
|
|
internal/language_features/built_in_objects.go#L58
string `Values` has 4 occurrences, make it a constant (goconst)
|
|
internal/language_features/built_in_objects.go#L64
string `Chart` has 3 occurrences, make it a constant (goconst)
|
|
internal/util/points.go#L17
var-naming: don't use underscores in Go names; func parameter range_ should be range (revive)
|
|
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
|
This job succeeded
Loading