stacklok / minder / 10508132634

53%

Build Type

push

github

Committed by web-flow

Commit Message

Use structpb.Value as internal storage of Property (#4250)

There are several reasons to use structpb.Value over `any`:
- NewValue contains pretty nice code that can e.g. serialize a []byte slice base64 encoded
- Returning the value as any actually does checks, for example for over/underflow for numerical values and returns a NaN in that case
- there is another draft PR that proposes to use structpb.Struct in the general entity protobuf message which would make the value easily usable should we need to

Related: #4171

Run Details

20 of 35 new or added lines in 2 files covered. (57.14%)

3 existing lines in 2 files now uncovered.

12808 of 23844 relevant lines covered (53.72%)

45.91 hits per line