feat(vscode:webview): implement dimming and highlighting of the relat… #540
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/vm/vm.go#L12
package comment is detached; there should be no blank lines between it and the package statement (golint)
|
lint:
internal/vm/vm.go#L51
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
lint:
internal/vm/vm.go#L43
error returned from interface method should be wrapped: sig: func (github.com/nevalang/neva/internal/vm.Loader).Load(ctx context.Context, path string) (bb []byte, err error) (wrapcheck)
|
lint:
internal/runtime/func_runner.go#L43
should not use basic type string as key in context.WithValue (golint)
|
lint:
internal/runtime/connector.go#L123
(connector).broadcast - result 0 (error) is always nil (unparam)
|
lint:
internal/runtime/program.go#L62
err113: do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"port not found by name: %v\", name)" (goerr113)
|
lint:
internal/runtime/connector.go#L203
S1023: redundant `return` statement (gosimple)
|
lint:
internal/runtime/func_runner.go#L30
named return "err" with type "error" found (nonamedreturns)
|
lint:
internal/runtime/runtime.go#L46
named return "code" with type "int" found (nonamedreturns)
|
lint:
internal/runtime/connector.go#L37
variable name 's' is too short for the scope of its usage (varnamelen)
|
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/
|