Astavie
Astavie
So, how's progress on this feature?
I was thinking of working on this issue, but I see `c-accepted` has been removed. What is the status on this proposal? Also, wouldn't `createProperty` also need a name argument...
The new adjacently tagged enum code really put a wrench into this. It effectively now serializes tags into a unit enum like: ```rust enum EnumTag { A, B, // ......
@dtolnay is making a new base serialization method a good solution here? It could even expand enum tags to more than booleans and integers later on and add support for...
It not necessarily an obstacle, but it does mean I need to do a slight rework of the (de)serialization of tags. Your implementation suggestion actually makes a lot of sense,...
Alright, that should be up-to-date again now
I've now done most of the simple changes. The rest of the requested changes will be completed on a later date
@xamgore There still needs to be support for different type of integer tags (see https://github.com/serde-rs/serde/pull/2525#discussion_r1285288903). This requires a bit of a refactor on top of the refactor i've already done....
Support for marking specific integer types has been added. Specifically: `i8`, `i16`, `i32`, `i64`, `u8`, `u16`, `u32`, `u64` Rules for unmarked integers: - If all variants are unmarked integers, the...
@jenr24-architect iirc, the only work required now are moving the highlight spot with the errors mentioned by @dtolnay above, though they also mentioned not having reviewed the deserialization code yet