fields for buildkite and ymal #15154
reviewdog [golangci] report
reported by reviewdog 🐶
Findings (62)
compiler/types/pipeline/test_report.go|1 col 1| Missed header for check (goheader)
compiler/types/pipeline/test_report_test.go|1 col 1| Missed header for check (goheader)
compiler/types/pipeline/container.go|51 col 33| tag is not aligned, should be: json:"test_report,omitempty" yaml:"test_report,omitempty" (tagalign)
compiler/types/pipeline/test_report.go|16 col 24| tag is not aligned, should be: yaml:"results,omitempty" json:"results,omitempty" (tagalign)
storage/minio/download.go|50 col 54| G115: integer overflow conversion int64 -> uint64 (gosec)
storage/minio/download.go|63 col 79| G115: integer overflow conversion int64 -> uint64 (gosec)
storage/minio/minio.go|22 col 6| exported: type name will be used as minio.MinioClient by other packages, and that stutters; consider calling this Client (revive)
storage/minio/minio.go|92 col 6| func pingBucket
is unused (unused)
storage/minio/stat_object.go|17 col 122| non-wrapping format verb for fmt.Errorf. Use %w
to format errors (errorlint)
storage/minio/bucket_exists.go|4 col 1| File is not properly formatted (gci)
storage/minio/create_bucket.go|4 col 1| File is not properly formatted (gci)
storage/minio/create_bucket.go|6 col 1| File is not properly formatted (gci)
storage/minio/upload.go|9 col 1| Comment should end in a period (godot)
storage/minio/bucket_exists.go|1 col 1| Missed header for check (goheader)
storage/minio/minio.go|37 col 2| ineffectual assignment to useSSL (ineffassign)
storage/minio/download.go|29 col 32| retrive
is a misspelling of retrieve
(misspell)
storage/minio/create_bucket.go|13 col 5| SA4022: the address of a variable cannot be nil (staticcheck)
storage/minio/download.go|95 col 1| unnecessary trailing newline (whitespace)
storage/minio/delete_bucket_test.go|55 col 1| unnecessary trailing newline (whitespace)
storage/minio/delete_bucket_test.go|86 col 1| unnecessary trailing newline (whitespace)
storage/minio/bucket_exists.go|16 col 2| return statements should not be cuddled if block has more than two lines (wsl)
storage/minio/create_bucket.go|22 col 2| assignments should only be cuddled with other assignments (wsl)
storage/minio/create_bucket.go|29 col 3| return statements should not be cuddled if block has more than two lines (wsl)
storage/minio/create_bucket.go|23 col 2| only one cuddle assignment allowed before if statement (wsl)
storage/minio/create_bucket.go|31 col 2| return statements should not be cuddled if block has more than two lines (wsl)
storage/minio/create_bucket.go|12 col 2| declarations should never be cuddled (wsl)
storage/minio/create_bucket.go|15 col 3| assignments should only be cuddled with other assignments (wsl)
storage/minio/get_bucket_lifecycle.go|14 col 2| only one cuddle assignment allowed before if statement (wsl)
storage/minio/get_bucket_lifecycle.go|13 col 2| assignments should only be cuddled with other assignments (wsl)
storage/minio/list_objects.go|15 col 2| ranges should only be cuddled with assignments used in the iteration (wsl)
storage/minio/list_objects.go|19 col 3| append only allowed to cuddle with appended value (wsl)
storage/minio/minio.go|112 col 89| block should not start with a whitespace (wsl)
storage/minio/minio.go|89 col 1| block should not end with a whitespace (or comment) (wsl)
storage/context.go|61 col 9| context-keys-type: should not use basic type string as key in context.WithValue (revive)
storage/context.go|10 col 1| Comment should end in a period (godot)
storage/context.go|18 col 1| Comment should end in a period (godot)
storage/storage.go|43 col 2| expressions should not be cuddled with blocks (wsl)
api/types/test-report/test_case.go|32 col 10| string ...
has 3 occurrences, make it a constant (goconst)
api/types/test-report/performance_results.go|1 col 9| var-naming: don't use an underscore in package name (revive)
api/types/test-report/test_run.go|1 col 9| var-naming: don't use an underscore in package name (revive)
api/types/test-report/test_run_system_attributes.go|1 col 9| var-naming: don't use an underscore in package name (revive)
api/types/test-report/test_suite.go|19 col 2| ST1003: struct field StartTs should be StartTS (stylecheck)
api/types/test-report/test_suite.go|324 col 21| ST1003: method SetStartTs should be SetStartTS (stylecheck)
api/types/test-report/test_suite.go|337 col 21| ST1003: method GetStartTs should be GetStartTS (stylecheck)
compiler/native/expand.go|396 col 33| string golang
has 3 occurrences, make it a constant (goconst)
compiler/native/compile_test.go|2449 col 13| SA1019: github.String is deprecated: use Ptr instead. (staticcheck)
compiler/native/compile_test.go|2450 col 13| SA1019: github.String is deprecated: use Ptr instead. (staticcheck)
scm/github/app_permissions.go|62 col 20| SA1019: github.String is deprecated: use Ptr instead. (staticcheck)
scm/github/deployment.go|180 col 21| SA1019: github.Bool is deprecated: use Ptr instead. (staticcheck)
scm/github/repo.go|209 col 11| SA1019: github.Bool is deprecated: use Ptr instead. (staticcheck)
scm/github/repo.go|283 col 11| SA1019: github.Bool is deprecated: use Ptr instead. (staticcheck)
scm/github/github_client_test.go|76 col 9| SA1019: github.Int64 is deprecated: use Ptr instead. (staticcheck)
scm/github/github_client_test.go|90 col 9| SA1019: github.Int64 is deprecated: use Ptr instead. (staticcheck)
scm/github/github_client_test.go|104 col 9| SA1019: github.Int64 is deprecated: use Ptr instead. (staticcheck)
compiler/types/yaml/buildkite/test_report.go|10 col 23| tag is not aligned, should be: yaml:"results,omitempty" json:"results,omitempty" (tagalign)
compiler/types/yaml/buildkite/test_report.go|27 col 24| tag is not aligned, should be: yaml:"results,omitempty" json:"results,omitempty" (tagalign)
api/admin/storage.go|312 col 2| if statements should only be cuddled with assignments (wsl)
api/admin/storage.go|390 col 2| if statements should only be cuddled with assignments (wsl)
api/admin/storage.go|396 col 2| only one cuddle assignment allowed before if statement (wsl)
api/admin/storage.go|385 col 2| if statements should only be cuddled with assignments (wsl)
api/admin/storage.go|65 col 2| expressions should not be cuddled with blocks (wsl)
storage/minio/minio.go|111 col 1| directive //nolint:revive // ignore returning unexported client
is unused for linter "revive" (nolintlint)
Filtered Findings (0)
Annotations
Check failure on line 1 in compiler/types/pipeline/test_report.go
github-actions / golangci
[golangci] compiler/types/pipeline/test_report.go#L1
Missed header for check (goheader)
Raw output
compiler/types/pipeline/test_report.go:1:1: Missed header for check (goheader)
package pipeline
^
Check failure on line 1 in compiler/types/pipeline/test_report_test.go
github-actions / golangci
[golangci] compiler/types/pipeline/test_report_test.go#L1
Missed header for check (goheader)
Raw output
compiler/types/pipeline/test_report_test.go:1:1: Missed header for check (goheader)
package pipeline
^
Check failure on line 51 in compiler/types/pipeline/container.go
github-actions / golangci
[golangci] compiler/types/pipeline/container.go#L51
tag is not aligned, should be: json:"test_report,omitempty" yaml:"test_report,omitempty" (tagalign)
Raw output
compiler/types/pipeline/container.go:51:33: tag is not aligned, should be: json:"test_report,omitempty" yaml:"test_report,omitempty" (tagalign)
TestReport TestReport `json:"test_report,omitempty" yaml:"test_report,omitempty"`
^
Check failure on line 16 in compiler/types/pipeline/test_report.go
github-actions / golangci
[golangci] compiler/types/pipeline/test_report.go#L16
tag is not aligned, should be: yaml:"results,omitempty" json:"results,omitempty" (tagalign)
Raw output
compiler/types/pipeline/test_report.go:16:24: tag is not aligned, should be: yaml:"results,omitempty" json:"results,omitempty" (tagalign)
Results []string `yaml:"results,omitempty" json:"results,omitempty"`
^
Check failure on line 50 in storage/minio/download.go
github-actions / golangci
[golangci] storage/minio/download.go#L50
G115: integer overflow conversion int64 -> uint64 (gosec)
Raw output
storage/minio/download.go:50:54: G115: integer overflow conversion int64 -> uint64 (gosec)
logrus.Infof("%s to download", humanize.Bytes(uint64(objInfo.Size)))
^
Check failure on line 63 in storage/minio/download.go
github-actions / golangci
[golangci] storage/minio/download.go#L63
G115: integer overflow conversion int64 -> uint64 (gosec)
Raw output
storage/minio/download.go:63:79: G115: integer overflow conversion int64 -> uint64 (gosec)
logrus.Infof("downloaded %s to %s on local filesystem", humanize.Bytes(uint64(stat.Size())), filename)
^
Check failure on line 22 in storage/minio/minio.go
github-actions / golangci
[golangci] storage/minio/minio.go#L22
exported: type name will be used as minio.MinioClient by other packages, and that stutters; consider calling this Client (revive)
Raw output
storage/minio/minio.go:22:6: exported: type name will be used as minio.MinioClient by other packages, and that stutters; consider calling this Client (revive)
type MinioClient struct {
^
Check failure on line 92 in storage/minio/minio.go
github-actions / golangci
[golangci] storage/minio/minio.go#L92
func `pingBucket` is unused (unused)
Raw output
storage/minio/minio.go:92:6: func `pingBucket` is unused (unused)
func pingBucket(c *MinioClient, bucket string) error {
^
Check failure on line 17 in storage/minio/stat_object.go
github-actions / golangci
[golangci] storage/minio/stat_object.go#L17
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
Raw output
storage/minio/stat_object.go:17:122: non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
return nil, fmt.Errorf("unable to get object info %s from bucket %s: %v", object.ObjectName, object.Bucket.BucketName, err)
^
Check failure on line 4 in storage/minio/bucket_exists.go
github-actions / golangci
[golangci] storage/minio/bucket_exists.go#L4
File is not properly formatted (gci)
Raw output
storage/minio/bucket_exists.go:4:1: File is not properly formatted (gci)
"context"
^
Check failure on line 4 in storage/minio/create_bucket.go
github-actions / golangci
[golangci] storage/minio/create_bucket.go#L4
File is not properly formatted (gci)
Raw output
storage/minio/create_bucket.go:4:1: File is not properly formatted (gci)
"context"
^
Check failure on line 6 in storage/minio/create_bucket.go
github-actions / golangci
[golangci] storage/minio/create_bucket.go#L6
File is not properly formatted (gci)
Raw output
storage/minio/create_bucket.go:6:1: File is not properly formatted (gci)
"github.com/minio/minio-go/v7"
^
Check failure on line 9 in storage/minio/upload.go
github-actions / golangci
[golangci] storage/minio/upload.go#L9
Comment should end in a period (godot)
Raw output
storage/minio/upload.go:9:1: Comment should end in a period (godot)
// Upload uploads an object to a bucket in MinIO.ts
^
Check failure on line 1 in storage/minio/bucket_exists.go
github-actions / golangci
[golangci] storage/minio/bucket_exists.go#L1
Missed header for check (goheader)
Raw output
storage/minio/bucket_exists.go:1:1: Missed header for check (goheader)
package minio
^
Check failure on line 37 in storage/minio/minio.go
github-actions / golangci
[golangci] storage/minio/minio.go#L37
ineffectual assignment to useSSL (ineffassign)
Raw output
storage/minio/minio.go:37:2: ineffectual assignment to useSSL (ineffassign)
useSSL := true
^
Check failure on line 29 in storage/minio/download.go
github-actions / golangci
[golangci] storage/minio/download.go#L29
`retrive` is a misspelling of `retrieve` (misspell)
Raw output
storage/minio/download.go:29:32: `retrive` is a misspelling of `retrieve` (misspell)
// c.Logger.Errorf("unable to retrive object %s", object.ObjectName)
^
Check failure on line 13 in storage/minio/create_bucket.go
github-actions / golangci
[golangci] storage/minio/create_bucket.go#L13
SA4022: the address of a variable cannot be nil (staticcheck)
Raw output
storage/minio/create_bucket.go:13:5: SA4022: the address of a variable cannot be nil (staticcheck)
if &bucket.Options == nil {
^
Check failure on line 95 in storage/minio/download.go
github-actions / golangci
[golangci] storage/minio/download.go#L95
unnecessary trailing newline (whitespace)
Raw output
storage/minio/download.go:95:1: unnecessary trailing newline (whitespace)
}
^
Check failure on line 55 in storage/minio/delete_bucket_test.go
github-actions / golangci
[golangci] storage/minio/delete_bucket_test.go#L55
unnecessary trailing newline (whitespace)
Raw output
storage/minio/delete_bucket_test.go:55:1: unnecessary trailing newline (whitespace)
}
^
Check failure on line 86 in storage/minio/delete_bucket_test.go
github-actions / golangci
[golangci] storage/minio/delete_bucket_test.go#L86
unnecessary trailing newline (whitespace)
Raw output
storage/minio/delete_bucket_test.go:86:1: unnecessary trailing newline (whitespace)
}
^
Check failure on line 16 in storage/minio/bucket_exists.go
github-actions / golangci
[golangci] storage/minio/bucket_exists.go#L16
return statements should not be cuddled if block has more than two lines (wsl)
Raw output
storage/minio/bucket_exists.go:16:2: return statements should not be cuddled if block has more than two lines (wsl)
return exists, nil
^
Check failure on line 22 in storage/minio/create_bucket.go
github-actions / golangci
[golangci] storage/minio/create_bucket.go#L22
assignments should only be cuddled with other assignments (wsl)
Raw output
storage/minio/create_bucket.go:22:2: assignments should only be cuddled with other assignments (wsl)
err := c.client.MakeBucket(ctx, bucket.BucketName, opts)
^
Check failure on line 29 in storage/minio/create_bucket.go
github-actions / golangci
[golangci] storage/minio/create_bucket.go#L29
return statements should not be cuddled if block has more than two lines (wsl)
Raw output
storage/minio/create_bucket.go:29:3: return statements should not be cuddled if block has more than two lines (wsl)
return err
^
Check failure on line 23 in storage/minio/create_bucket.go
github-actions / golangci
[golangci] storage/minio/create_bucket.go#L23
only one cuddle assignment allowed before if statement (wsl)
Raw output
storage/minio/create_bucket.go:23:2: only one cuddle assignment allowed before if statement (wsl)
if err != nil {
^
Check failure on line 31 in storage/minio/create_bucket.go
github-actions / golangci
[golangci] storage/minio/create_bucket.go#L31
return statements should not be cuddled if block has more than two lines (wsl)
Raw output
storage/minio/create_bucket.go:31:2: return statements should not be cuddled if block has more than two lines (wsl)
return nil
^