qri-io / starlib

qri's standard library for starlark
MIT License
113 stars 29 forks source link

bug: Marshal and Unmarshal don't support bytes #172

Open HarikrishnanBalagopal opened 1 year ago

HarikrishnanBalagopal commented 1 year ago

Bug

The function Unmarshal doesn't have a case for handling the starlark.Bytes type. https://github.com/qri-io/starlib/blob/7fb7ff9ec804789a6dd0c1470abc5f07ba4a0c46/util/util.go#L27-L158

Context

$ starlark
Welcome to Starlark (go.starlark.net)
>>> bytes([1,2])
b"\x01\x02"