Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
[CHATOPS:HELP] ChatOps commands.
|
|
/format |
|
[FORMAT] Updating license headers and formatting go codes triggered by Matts966. |
Deploying vald with
|
| Latest commit: |
469364f
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://12648772.vald.pages.dev |
| Branch Preview URL: | https://feature-grpc-common-any-args.vald.pages.dev |
|
[FORMAT] Failed to format. |
|
/format |
|
[FORMAT] Updating license headers and formatting go codes triggered by Matts966. |
|
/format |
|
[FORMAT] Updating license headers and formatting go codes triggered by Matts966. |
|
[FORMAT] Failed to format. |
There was a problem hiding this comment.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
golangci
🚫 [golangci] reported by reviewdog 🐶
Magic number: 11, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28238 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 12, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28257 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 13, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28276 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 14, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28295 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 15, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28314 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 16, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28333 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 17, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28352 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 18, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28371 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 19, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28390 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 20, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28409 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 21, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28428 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 22, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28439 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 23, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28450 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 24, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28461 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 25, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28472 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 26, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28483 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 27, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28494 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
Magic number: 28, in detected (mnd)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 28505 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
ST1006: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (staticcheck)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 2864 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
ST1006: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (staticcheck)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 2885 in e7bf077
🚫 [golangci] reported by reviewdog 🐶
unnecessary conversion (unconvert)
vald/apis/grpc/v1/payload/payload_vtproto.pb.go
Line 6634 in e7bf077
|
/format |
|
[FORMAT] Updating license headers and formatting go codes triggered by Matts966. |
Signed-off-by: Vdaas CI <vald@vdaas.org>
Signed-off-by: Vdaas CI <vald@vdaas.org>
ee05196 to
469364f
Compare
| @@ -19398,6 +19362,40 @@ func (m *Search_Config) UnmarshalVT(dAtA []byte) error { | |||
| break | |||
| } | |||
| } | |||
| case 13: | |||
There was a problem hiding this comment.
🚫 [golangci] reported by reviewdog 🐶
19365-19398 lines are duplicate of apis/grpc/v1/payload/payload_vtproto.pb.go:20617-20650 (dupl)
| @@ -20628,6 +20614,40 @@ func (m *Insert_Config) UnmarshalVT(dAtA []byte) error { | |||
| break | |||
| } | |||
| } | |||
| case 4: | |||
There was a problem hiding this comment.
🚫 [golangci] reported by reviewdog 🐶
20617-20650 lines are duplicate of apis/grpc/v1/payload/payload_vtproto.pb.go:31547-31580 (dupl)
| @@ -31610,6 +31544,40 @@ func (m *Search_Config) UnmarshalVTUnsafe(dAtA []byte) error { | |||
| break | |||
| } | |||
| } | |||
| case 13: | |||
There was a problem hiding this comment.
🚫 [golangci] reported by reviewdog 🐶
31547-31580 lines are duplicate of apis/grpc/v1/payload/payload_vtproto.pb.go:32807-32840 (dupl)
| @@ -32848,6 +32804,40 @@ func (m *Insert_Config) UnmarshalVTUnsafe(dAtA []byte) error { | |||
| break | |||
| } | |||
| } | |||
| case 4: | |||
There was a problem hiding this comment.
🚫 [golangci] reported by reviewdog 🐶
32807-32840 lines are duplicate of apis/grpc/v1/payload/payload_vtproto.pb.go:19365-19398 (dupl)
Summary
google.protobuf.Anyoptions to Search and Insert payload configsVerification
make proto/all(Buf generation succeeded; repository replacements were applied in smaller batches because macOSxargsexceeded its command-line limit)make licenseBuild protobufcheck passed on the rebased headStacked PR
#3574 is based on
feature/grpc/common-any-argsand contains the ZVec-specific messages and implementation.