Skip to content

Implement support for DeserializeSeed through Options#360

Merged
juntyr merged 2 commits into
ron-rs:masterfrom
juntyr:359-deserialize-seed
Jan 28, 2022
Merged

Implement support for DeserializeSeed through Options#360
juntyr merged 2 commits into
ron-rs:masterfrom
juntyr:359-deserialize-seed

Conversation

@juntyr

@juntyr juntyr commented Jan 26, 2022

Copy link
Copy Markdown
Member

Fixes #359

  • I've included my change in CHANGELOG.md

@juntyr juntyr force-pushed the 359-deserialize-seed branch from acf5b83 to 0bf7802 Compare January 26, 2022 13:12
@juntyr

juntyr commented Jan 26, 2022

Copy link
Copy Markdown
Member Author

?r @torkleyy Thought this might be a nice quick fix for a tea break. I decided to add the DeserializeSeed variants only to the new Options API instead of further cluttering ron::de.

@torkleyy torkleyy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!
Don't have much time to spend on GitHub lately, sorry.

@juntyr juntyr merged commit 23cefc9 into ron-rs:master Jan 28, 2022
@juntyr juntyr deleted the 359-deserialize-seed branch January 28, 2022 08:45
torkleyy pushed a commit to torkleyy/ron that referenced this pull request Jun 6, 2022
@torkleyy torkleyy mentioned this pull request Jun 6, 2022
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support DeserializeSeed

2 participants