Capigraph, UI update #334
kleineshertzAnnotations
10 errors and 1 warning
|
build:
pkg/capigraph/svg_test.go#L18
use of `fmt.Printf` forbidden by pattern `^(fmt\.Print(|f|ln)|print|println)$` (forbidigo)
|
|
build:
pkg/capigraph/svg_test.go#L25
use of `fmt.Printf` forbidden by pattern `^(fmt\.Print(|f|ln)|print|println)$` (forbidigo)
|
|
build:
pkg/capigraph/svg_test.go#L32
use of `fmt.Printf` forbidden by pattern `^(fmt\.Print(|f|ln)|print|println)$` (forbidigo)
|
|
build:
pkg/capigraph/layer_mx_permutator_test.go#L79
unused-parameter: parameter 'i' seems to be unused, consider removing or renaming it as _ (revive)
|
|
build:
pkg/capigraph/layer_mx_permutator.go#L91
empty-block: this block is empty, you can remove it (revive)
|
|
build:
pkg/capigraph/fonts.go#L107
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
|
|
build:
pkg/capigraph/layering.go#L42
cognitive-complexity: function buildLayerMap has cognitive complexity 53 (> max enabled 50) (revive)
|
|
build:
pkg/capigraph/layer_permutator.go#L50
cognitive-complexity: function (*LayerPermutator).swapPermutationByIdx has cognitive complexity 86 (> max enabled 50) (revive)
|
|
build:
pkg/capigraph/node_def.go#L190
range: should omit 2nd value from range; this loop is equivalent to `for i := range ...` (revive)
|
|
build:
pkg/capigraph/layer_mx_permutator_test.go#L22
comment-spacings: no space between comment delimiter and comment text (revive)
|
|
build
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|