Skip to content

Add ygnmi validator #144

Add ygnmi validator

Add ygnmi validator #144

Triggered via pull request November 10, 2023 03:15
@wenovuswenovus
synchronize #93
ygnmi-for-ygot
Status Failure
Total duration 38s
Artifacts

go.yml

on: pull_request
Matrix: go / Build and Test
Fit to window
Zoom out
Zoom in

Annotations

13 errors
go / Build and Test (stable)
Process completed with exit code 1.
go / Static Analysis
Process completed with exit code 1.
go / Static Analysis: openconfig-ci/main.go#L15
should have a package comment
go / Static Analysis: post_results/misc-checks.go#L115
should replace filesReachedCount += 1 with filesReachedCount++
go / Static Analysis: post_results/misc-checks.go#L129
should replace ocVersionChangedCount += 1 with ocVersionChangedCount++
go / Static Analysis: post_results/misc-checks.go#L140
should replace ocVersionChangedCount += 1 with ocVersionChangedCount++
go / Static Analysis: cmd/check.go#L15
should have a package comment
go / Static Analysis: yangutil/yangutil.go#L23
exported function GetAllYANGFiles should have comment or be unexported
go / Static Analysis: yangutil/yangutil.go#L15
package comment should be of the form "Package yangutil ..."
go / Static Analysis: validators/misc-checks/ocversion/ocversion.go#L15
should have a package comment
go / Static Analysis: util/parseutil.go#L32
don't use ALL_CAPS in Go names; use CamelCase
go / Static Analysis: openconfig-ci/ocdiff/ocdiff.go#L158
if block ends with a return statement, so drop this else and outdent its block
go / Build and Test (oldstable)
Process completed with exit code 1.