refactor: add validation to apf #477
Annotations
10 errors and 2 warnings
golangci-lint:
pkg/apf/vaildator.go#L18
[golangci] reported by reviewdog 🐶
return with no blank line before (nlreturn)
Raw Output:
pkg/apf/vaildator.go:18:2: return with no blank line before (nlreturn)
return len(data) >= 5+serviceLen
^
|
golangci-lint:
pkg/apf/vaildator.go#L39
[golangci] reported by reviewdog 🐶
return with no blank line before (nlreturn)
Raw Output:
pkg/apf/vaildator.go:39:2: return with no blank line before (nlreturn)
return false
^
|
golangci-lint:
pkg/apf/vaildator.go#L37
[golangci] reported by reviewdog 🐶
return with no blank line before (nlreturn)
Raw Output:
pkg/apf/vaildator.go:37:3: return with no blank line before (nlreturn)
return len(data) >= 14+globalReqLen+addrLen
^
|
golangci-lint:
pkg/apf/processor.go#L39
[golangci] reported by reviewdog 🐶
if statements should only be cuddled with assignments (wsl)
Raw Output:
pkg/apf/processor.go:39:3: if statements should only be cuddled with assignments (wsl)
if ValidateChannelOpenConfirmation(data) {
^
|
golangci-lint:
pkg/apf/processor.go#L44
[golangci] reported by reviewdog 🐶
if statements should only be cuddled with assignments (wsl)
Raw Output:
pkg/apf/processor.go:44:3: if statements should only be cuddled with assignments (wsl)
if ValidateChannelOpenFailure(data) {
^
|
golangci-lint:
pkg/apf/processor.go#L49
[golangci] reported by reviewdog 🐶
if statements should only be cuddled with assignments (wsl)
Raw Output:
pkg/apf/processor.go:49:3: if statements should only be cuddled with assignments (wsl)
if ValidateChannelClose(data) {
^
|
golangci-lint:
pkg/apf/vaildator.go#L17
[golangci] reported by reviewdog 🐶
assignments should only be cuddled with other assignments (wsl)
Raw Output:
pkg/apf/vaildator.go:17:2: assignments should only be cuddled with other assignments (wsl)
serviceLen := int(binary.BigEndian.Uint32(data[1:5]))
^
|
golangci-lint:
pkg/apf/vaildator.go#L26
[golangci] reported by reviewdog 🐶
assignments should only be cuddled with other assignments (wsl)
Raw Output:
pkg/apf/vaildator.go:26:2: assignments should only be cuddled with other assignments (wsl)
globalReqLen := int(binary.BigEndian.Uint32(data[1:5]))
^
|
golangci-lint:
pkg/apf/vaildator.go#L27
[golangci] reported by reviewdog 🐶
only one cuddle assignment allowed before if statement (wsl)
Raw Output:
pkg/apf/vaildator.go:27:2: only one cuddle assignment allowed before if statement (wsl)
if len(data) < 5+globalReqLen+1 {
^
|
golangci-lint
reviewdog: Too many results (annotations) in diff.
You may miss some annotations due to GitHub limitation for annotation created by logging command.
Please check GitHub Actions log console to see all results.
Limitation:
- 10 warning annotations and 10 error annotations per step
- 50 annotations per job (sum of annotations from all the steps)
- 50 annotations per run (separate from the job annotations, these annotations aren't created by users)
Source: https://github.com/orgs/community/discussions/26680#discussioncomment-3252835
|
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
golangci-lint
Input 'fail_on_error' has been deprecated with message: Deprecated, use `fail_level` instead.
|
Loading