serde-xml-rs icon indicating copy to clipboard operation
serde-xml-rs copied to clipboard

Support seq and maps (rebase of #142, refactor of #129)

Open joemat opened this issue 3 years ago • 4 comments

Based on #142 and #129 this PR adds support to serialize maps and vectors. See description of #142 for details.

This is just a rebased version of #142 which fixes the merge conflict.

Kudos to @adrianbenavides and @Niedzwiedzw for their work on this.

joemat avatar Apr 12 '21 18:04 joemat

Closes #135.

ghost avatar Apr 12 '21 18:04 ghost

FYI, I've contributed to and find this project useful and so proposed we move it to a group: #155.

ghost avatar Apr 12 '21 18:04 ghost

Hi, I noticed this awesome job. But it seems to have been around for a while and has some conflicts from the master branch.

Would you complete this Pull Request? Or if possible I can also take over and resolve the conflicting part.

XiaoXiaoSN avatar Feb 16 '22 11:02 XiaoXiaoSN

Hi, I noticed this awesome job. But it seems to have been around for a while and has some conflicts from the master branch.

Would you complete this Pull Request? Or if possible I can also take over and resolve the conflicting part.

FTR you did the rebase here #170, thanks.

victorpaleologue avatar Mar 17 '22 14:03 victorpaleologue

Should this be closed because of #173? #170 says that #173 merges the changes from some other changes including this one.

wt avatar Jan 09 '23 08:01 wt

yes, makes sense to close this one then

joemat avatar Jan 22 '23 09:01 joemat