search

davidhewitt / pythonize

MIT License
198 stars 28 forks source link

serialize set and frozenset #72

Closed davidhewitt closed 1 month ago

davidhewitt commented 1 month ago

Long overdue merge of #39

Thanks @LilyFoote and sorry this took so long to actually see across the line. I went for a slightly different approach to avoid the potential of infinite iterators and only accept existing sequences plus the new set/frozenset options.

Closes #39

davidhewitt commented 1 month ago

Immediately upon hitting the flaky test error due to sets being unordered and conflicting with an inhomogeneous tuple of type (String, i32), I've narrowed this so that we only allow deserializing into homogeneous sequence containers like Vec, and not tuples.