Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(#151): renaming Deserializer::visit_* to Deserializer::deserialize_* #170

Merged
merged 2 commits into from
Oct 31, 2015

Conversation

oli-obk
Copy link
Member

@oli-obk oli-obk commented Oct 8, 2015

serde-json PR @ serde-rs/json#13

@erickt
Copy link
Member

erickt commented Oct 18, 2015

Looks good, other than the test failing. Could you update it?

@oli-obk oli-obk force-pushed the master branch 3 times, most recently from e05737a to 048d0ca Compare October 19, 2015 11:10
@oli-obk
Copy link
Member Author

oli-obk commented Oct 19, 2015

test passes now

@erickt
Copy link
Member

erickt commented Oct 19, 2015

@oli-obk: Shucks, just broke it on you again :(

erickt added a commit that referenced this pull request Oct 31, 2015
fix(#151): renaming Deserializer::visit_* to Deserializer::deserialize_*
@erickt erickt merged commit 4bdabea into serde-rs:master Oct 31, 2015
rubdos pushed a commit to rubdos/serde that referenced this pull request Jun 20, 2017
Add formatting options for Complex

This adds LowerExp and UpperExp traits for Complex, taking precision into account. Fixes serde-rs#148
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants