update review tips #27
liguozhongci.yml
on: push
Matrix: codeql / Analyze
UI tests
3m 0s
Matrix: Build Prometheus for all architectures
Go tests
38m 5s
Go tests on Windows
16m 10s
golangci-lint
5m 15s
Go tests with previous Go version
13m 21s
Mixins tests
3m 30s
fuzzing
/
Fuzzing
Matrix: Build Prometheus for common architectures
Publish UI on npm Registry
2m 39s
Publish main branch artifacts
0s
Publish release artefacts
0s
Annotations
12 errors
|
golangci-lint:
notifier/notifier.go#L628
File is not `gofumpt`-ed with `-extra` (gofumpt)
|
|
golangci-lint:
web/api/v1/api.go#L1167
assignOp: replace `res.DroppedTargetCounts[job] = res.DroppedTargetCounts[job] + 1` with `res.DroppedTargetCounts[job]++` (gocritic)
|
|
golangci-lint:
web/api/v1/api.go#L1177
commentFormatting: put a space between `//` and comment text (gocritic)
|
|
golangci-lint:
web/api/v1/api.go#L1179
File is not `gofumpt`-ed with `-extra` (gofumpt)
|
|
golangci-lint:
web/api/v1/api_test.go#L1566
commentFormatting: put a space between `//` and comment text (gocritic)
|
|
golangci-lint:
web/api/v1/api_test.go#L1570
File is not `gofumpt`-ed with `-extra` (gofumpt)
|
|
golangci-lint:
web/api/v1/api_test.go#L1601
File is not `gofumpt`-ed with `-extra` (gofumpt)
|
|
golangci-lint:
web/api/v1/api_test.go#L1615
File is not `gofumpt`-ed with `-extra` (gofumpt)
|
|
golangci-lint
issues found
|
|
codeql / Analyze (go):
web/api/v1/api.go#L1167
assignOp: replace `res.DroppedTargetCounts[job] = res.DroppedTargetCounts[job] + 1` with `res.DroppedTargetCounts[job]++` (gocritic)
|
|
codeql / Analyze (go):
web/api/v1/api.go#L1177
commentFormatting: put a space between `//` and comment text (gocritic)
|
|
codeql / Analyze (go):
web/api/v1/api_test.go#L1566
commentFormatting: put a space between `//` and comment text (gocritic)
|
Artifacts
Produced during runtime
| Name | Size | |
|---|---|---|
|
artifact
Expired
|
1.35 GB |
|