cattrs icon indicating copy to clipboard operation
cattrs copied to clipboard

Register Iterable instead of Sequence for dumps?

Open wsanchez opened this issue 8 years ago • 1 comments

I was looking at the code and noticed that in converters.py:38, you use a Sequence where an Iterable would work for more object types.

Looking at _dump_seq, all it does it iterate over the given seq, so no other change should be needed.

wsanchez avatar Feb 25 '17 00:02 wsanchez

Why not? Try making the change and if the tests pass, submit a PR and I'll take a look :)

Tinche avatar Feb 25 '17 14:02 Tinche