jangko / msgpack4nim

MessagePack serializer/deserializer implementation for Nim / msgpack.org[Nim]
http://msgpack.org/
119 stars 21 forks source link

Make MsgStream.writeData() public #70

Closed snej closed 1 year ago

snej commented 1 year ago

Fixes #69

I assume this doesn’t need a unit test, because writeData is already called by other write methods so it’s already being tested.

snej commented 1 year ago

I think something’s wrong with your CI, since the only change in this PR is the addition of a * to make writeData public. I looked at some of the error messages but they don’t make sense to me and are unrelated to my change.

jangko commented 1 year ago

closing in favor of #71