Skip to content

Intial Commit Review. #2

Intial Commit Review.

Intial Commit Review. #2

Triggered via pull request March 4, 2024 20:28
@KevinJoinerKevinJoiner
synchronize #1
main
Status Failure
Total duration 18s
Artifacts

lint.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 7 warnings
lint: internal/codegen/convert/convert.go#L25
pattern convertTestFunc.tmpl: no matching files found (typecheck)
lint: pkg/vss/vss-convert-funcs_test.go#L907
907-976 lines are duplicate of `pkg/vss/vss-convert-funcs_test.go:977-1046` (dupl)
lint: pkg/vss/vss-convert-funcs_test.go#L977
977-1046 lines are duplicate of `pkg/vss/vss-convert-funcs_test.go:1134-1203` (dupl)
lint: pkg/vss/vss-convert-funcs_test.go#L1134
1134-1203 lines are duplicate of `pkg/vss/vss-convert-funcs_test.go:1204-1273` (dupl)
lint: pkg/vss/vss-convert-funcs_test.go#L1204
1204-1273 lines are duplicate of `pkg/vss/vss-convert-funcs_test.go:1495-1564` (dupl)
lint: pkg/vss/vss-convert-funcs_test.go#L1495
1495-1564 lines are duplicate of `pkg/vss/vss-convert-funcs_test.go:907-976` (dupl)
lint: pkg/vss/vss-convert-funcs_test.go#L22
22-84 lines are duplicate of `pkg/vss/vss-convert-funcs_test.go:85-148` (dupl)
lint: pkg/vss/vss-convert-funcs_test.go#L85
85-148 lines are duplicate of `pkg/vss/vss-convert-funcs_test.go:149-212` (dupl)
lint: pkg/vss/vss-convert-funcs_test.go#L149
149-212 lines are duplicate of `pkg/vss/vss-convert-funcs_test.go:213-276` (dupl)
lint: pkg/vss/vss-convert-funcs_test.go#L213
213-276 lines are duplicate of `pkg/vss/vss-convert-funcs_test.go:22-84` (dupl)
lint
Restore cache failed: Dependencies file is not found in /home/runner/work/model-garage/model-garage. Supported file pattern: go.sum
lint: internal/convert/convert.go#L41
exported: comment on exported function Float64toInt16 should be of the form "Float64toInt16 ..." (revive)
lint: pkg/vss/vss-convert-funcs.go#L1
package-comments: should have a package comment (revive)
lint: internal/codegen/codegen.go#L13
exported: exported const ClickhouseFileName should have comment (or a comment on this block) or be unexported (revive)
lint: internal/codegen/codegen.go#L5
blank-imports: a blank import should be only in a main or test package, or have a comment justifying it (revive)
lint: internal/codegen/clickhouse/clickhouse.go#L1
package-comments: should have a package comment (revive)
lint: internal/codegen/model/model.go#L1
package-comments: should have a package comment (revive)