Skip to content

Remove bingo

Remove bingo #1813

Triggered via pull request November 8, 2024 10:52
Status Failure
Total duration 3m 0s
Artifacts

pull_request_e2e.yml

on: pull_request
Matrix: e2e-tests
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 1 warning
e2e-tests (1.22): pkg/pipeline/transform/location/location_test.go#L131
commentFormatting: put a space between `//` and comment text (gocritic)
e2e-tests (1.22): pkg/pipeline/transform/location/location_test.go#L140
commentFormatting: put a space between `//` and comment text (gocritic)
e2e-tests (1.22): pkg/pipeline/transform/location/location_test.go#L144
commentFormatting: put a space between `//` and comment text (gocritic)
e2e-tests (1.22): cmd/apitodoc/main.go#L39
ifElseChain: rewrite if-else to switch statement (gocritic)
e2e-tests (1.22): pkg/utils/convert.go#L59
ifElseChain: rewrite if-else to switch statement (gocritic)
e2e-tests (1.22): pkg/utils/convert.go#L96
ifElseChain: rewrite if-else to switch statement (gocritic)
e2e-tests (1.22): pkg/pipeline/extract/timebased/filters.go#L105
assignOp: replace `currentValue = currentValue / float64(nItems)` with `currentValue /= float64(nItems)` (gocritic)
e2e-tests (1.22): pkg/pipeline/extract/timebased/timebased.go#L83
elseif: can replace 'else {if cond {}}' with 'else if cond {}' (gocritic)
e2e-tests (1.22): pkg/pipeline/extract/aggregate/aggregate.go#L152
elseif: can replace 'else {if cond {}}' with 'else if cond {}' (gocritic)
e2e-tests (1.22): pkg/pipeline/extract/aggregate/aggregates.go#L73
appendAssign: append result not assigned to the same slice (gocritic)
e2e-tests (1.22)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v3, actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/