Add testing for reconcile function. (#202) #1109
robshakir
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
go.yml
on: push
go
/
Static Analysis
7m 23s
go
/
staticcheck
7m 30s
Matrix: go / Build and Test
Annotations
10 errors
|
go / Static Analysis:
constants/const.go#L15
package comment should be of the form "Package constants ..."
|
|
go / Static Analysis:
server/server.go#L412
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
go / Static Analysis:
server/server.go#L138
type name will be used as server.ServerOpt by other packages, and that stutters; consider calling this Opt
|
|
go / Static Analysis:
cmd/ccli/ccli_test.go#L59
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
go / Static Analysis:
server/server.go#L167
comment on exported function WithRIBResolvedEntryHook should be of the form "WithRIBResolvedEntryHook ..."
|
|
go / Static Analysis:
server/server.go#L144
exported func WithPostChangeRIBHook returns unexported type *server.postChangeRibHook, which can be annoying to use
|
|
go / Static Analysis:
cmd/ccli/ccli_test.go#L59
parameter 'uri' seems to be unused, consider removing or renaming it as _
|
|
go / Static Analysis:
server/server.go#L169
exported func WithRIBResolvedEntryHook returns unexported type *server.resolvedEntryHook, which can be annoying to use
|
|
go / Static Analysis:
server/server.go#L196
exported func DisableRIBCheckFn returns unexported type *server.disableCheckFn, which can be annoying to use
|
|
go / Static Analysis:
server/server.go#L218
exported func WithVRFs returns unexported type *server.withVRFs, which can be annoying to use
|