add test for leader callbacks #2
thevilledevAnnotations
10 errors
|
Test:
gonsensus_test.go#L445
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
Test:
gonsensus_test.go#L449
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
Test:
gonsensus_test.go#L415
do not define dynamic errors, use wrapped static errors instead: "errors.New(\"simulated error\")" (err113)
|
|
Test:
gonsensus_test.go#L495
for loop can be changed to use an integer range (Go 1.22+) (intrange)
|
|
Test:
gonsensus_test.go#L447
return with no blank line before (nlreturn)
|
|
Test:
gonsensus_test.go#L458
variable name 'wg' is too short for the scope of its usage (varnamelen)
|
|
Test:
gonsensus_test.go#L459
expressions should not be cuddled with declarations or returns (wsl)
|
|
Test:
gonsensus_test.go#L462
assignments should only be cuddled with other assignments (wsl)
|
|
Test:
gonsensus_test.go#L463
only one cuddle assignment allowed before if statement (wsl)
|
|
Test:
gonsensus_test.go#L460
only one cuddle assignment allowed before go statement (wsl)
|