Add support for Mapping, Sequence, Set and FrozenSet #131
Comments
|
Seems related to #99 |
lovasoa
added a commit
that referenced
this issue
Apr 5, 2021
* Add support for Mapping, Sequence, Set and FrozenSet With this, most used collection types are supported in the dataclass. fixes #131 * stronger assertions in tests * v8.4.0 Co-authored-by: ophir <pere.jobs@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The goal here is to go a step further than #120 and add support for the other commonly used collection types.
Why do I even bother ?
the main goal here is to "reintroduce" the Immutable fields in the dataclass. Currently we are limited to
List, Dict, Tuple. This is mostly fine, butListandDictare mutable and give the overall "meh" result once associated withdataclass(frozen=true).Thus, among those in the list,
SequenceandMappingare my must have.SetandFrozenSetare here just for completness.Notes:
I'm not very sure how to handle the new fields needed to implement this.
The implementation is pretty straight forward as it's just a special case of
marshmallow.fields.List, by simply changing the collection type used in the_deserialize.But, just like the
Unionfield, should those be considered extras ?The text was updated successfully, but these errors were encountered: