fix: prevent forceful exit on invalid command output (#118) #194
on_push_pr.yaml
on: push
push-pr
/
...
/
Test binary compilation for all platforms:arch
push-pr
/
...
/
Run all static analysis checks
48s
push-pr
/
...
/
Run unit tests on *Nix
40s
push-pr
/
...
/
Run unit tests on Windows
28s
push-pr
/
...
/
Run integration tests on *Nix
41s
push-pr
/
...
/
Test binary compilation for all platforms:arch
1m 29s
Matrix: push-pr / test-build-fake-prerelease / Create MSI
Waiting for pending jobs
Annotations
10 errors and 1 warning
push-pr / static-analysis / Run all static analysis checks:
src/main.go#L142
cyclomatic complexity 14 of func `collectServiceCheck` is high (> 10) (gocyclo)
|
push-pr / static-analysis / Run all static analysis checks:
tests/jsonschema/jsonschema.go#L44
File is not `gofmt`-ed with `-s` (gofmt)
|
push-pr / static-analysis / Run all static analysis checks:
src/main.go#L254
`if err != nil` has complex nested blocks (complexity: 5) (nestif)
|
push-pr / static-analysis / Run all static analysis checks:
tests/jsonschema/jsonschema.go#L27
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"Error loading JSON schema, error: %v\", err)" (err113)
|
push-pr / static-analysis / Run all static analysis checks:
tests/jsonschema/jsonschema.go#L38
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"The output of the integration doesn't have expected JSON format\")" (err113)
|
push-pr / static-analysis / Run all static analysis checks:
src/main.go#L121
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"service checks file permissions are not restrictive enough. File permissions must be more strict than 0640. See documentation for details\")" (err113)
|
push-pr / static-analysis / Run all static analysis checks:
src/main.go#L129
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"failed to read config file: %s\", err.Error())" (err113)
|
push-pr / static-analysis / Run all static analysis checks:
src/main.go#L136
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"failed to parse config file: %s\", err.Error())" (err113)
|
push-pr / static-analysis / Run all static analysis checks:
src/main.go#L256
type assertion on error will fail on wrapped errors. Use errors.As to check for specific errors (errorlint)
|
push-pr / static-analysis / Run all static analysis checks:
src/main.go#L120
Magic number: 0o640, in <condition> detected (mnd)
|
push-pr / static-analysis / Run all static analysis checks
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|