Mingun

Results 662 comments of Mingun

@dtolnay , when you could look at this? This bugs becomes annoying...

@dtolnay, @oli-obk, can you give some feedback?

@dtolnay, @oli-obk, can this bugfix be merged?

Why you want to specify module? It seems that just customization of visibility will be enough for your use case. Maybe also add an ability to customize generated name.

PR #2558 should improve messages for deserialize -- errors will point to the content of `with` attribute.

Actually ability to deserialize `()` from empty sequence was removed in #839 (and ability to deserialize unit structs from empty sequences removed in #857). However, there is still some inconsistency....

> If I unterstand you correctly, you argue that `()` should never be deserialized from an empty sequence because it would be a type conversion? I would not say what...

I think, that instead of introducing new attribute for specifying an alias multiple `#[serde(tag)]` (and `#[serde(content)]`) attributes may be used

It seems that this issue should be moved to https://github.com/serde-rs/json