gbn+mailbox: cleanup & prefixed logger #398
main.yml
on: pull_request
build package and wasm
4m 7s
lint code
2m 4s
run itests
3m 2s
Matrix: run unit tests
Annotations
6 errors and 5 warnings
lint code:
gbn/gbn_conn.go#L269
expressions should not be cuddled with blocks (wsl)
|
lint code:
gbn/gbn_conn.go#L287
expressions should not be cuddled with blocks (wsl)
|
lint code:
gbn/gbn_conn.go#L70
found a struct that contains a context.Context field (containedctx)
|
lint code:
gbn/gbn_client.go#L165
type assertion must be checked (forcetypeassert)
|
lint code:
gbn/gbn_conn.go#L610
err113: do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"received unexpected message: %T\",\n\tmsg)" (goerr113)
|
lint code
Process completed with exit code 2.
|
run unit tests (unit)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-go@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
run unit tests (unit-race)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-go@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
lint code
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-go@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
run itests
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-go@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
build package and wasm
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-go@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|