Merge pull request #385 from nevalang/mod #521
emil14
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
Annotations
10 errors and 1 warning
|
lint:
internal/compiler/analyzer/const.go#L17
cognitive complexity 39 of func `(Analyzer).analyzeConst` is high (> 30) (gocognit)
|
|
lint:
internal/runtime/func_runner.go#L43
should not use basic type string as key in context.WithValue (golint)
|
|
lint:
internal/vm/vm.go#L12
package comment is detached; there should be no blank lines between it and the package statement (golint)
|
|
lint:
pkg/typesystem/resolver_test.go#L13
error strings should not be capitalized or end with punctuation or a newline (golint)
|
|
lint:
cmd/lsp/general_messages.go#L70
receiver name srv should be consistent with previous receiver name s for Server (golint)
|
|
lint:
cmd/lsp/general_messages.go#L76
receiver name srv should be consistent with previous receiver name s for Server (golint)
|
|
lint:
internal/interpreter/interpreter.go#L13
struct of size 128 bytes could be of size 120 bytes (maligned)
|
|
lint:
internal/runtime/connector.go#L123
(connector).broadcast - result 0 (error) is always nil (unparam)
|
|
lint:
internal/compiler/analyzer/component.go#L191
`(Analyzer).checkNodeUsage` - `scope` is unused (unparam)
|
|
lint:
internal/compiler/analyzer/component.go#L220
`(Analyzer).getReceiverType` - `nodesIfaces` is unused (unparam)
|
|
lint
The following actions uses node12 which is deprecated and will be forced to run on node16: 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/
|