Closed adrian-kong closed 2 years ago
@silverjam I think CI is currently enabled for this PR?
(also, I should be able to merge this this weekend, hopefully)
There are some merge conflicts though, maybe that's blocking CI.
@silverjam Sorry for the delay, yeah apparently it wasn't enabled, not sure what was up with that. Anyway I enabled CI and it looks like we just need to use serde::{Serialize, Deserialize}
@lkolbly Ok, builds and tests are passing locally for me -- looks like you'll need to approve the CI one more time
Thanks for getting this in @lkolbly!
This PR adds some of missing messages (before recent merges) and code-gen for serde serialize implementations, helpful for serializing iterators.
List of new messages:
Serialization outputs includes the class id and message id*