-
Notifications
You must be signed in to change notification settings - Fork 276
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
Update serde dependency to 0.9.0 #38
Conversation
This lets json-rpc build on stable (1.15) without needing serde_codegen
Thank you! Closes #16 |
core/src/lib.rs
Outdated
|
||
#[macro_use] extern crate log; | ||
extern crate serde; | ||
extern crate serde_json; | ||
#[macro_use] extern crate serde_derive; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
style: would prefer to keep #[macro_use]
links adjacent
tiny style nit but LGTM. Out of curiosity, how does this affect compile times? |
Running |
All good then, thanks! |
side note, we'll need quite substantial refactoring to switch |
When do you think you can release a version with this PR? Using a git dependency right now since I can't use serde 0.8.0 due to another dependency. It works for development but I can't do an actual release to crates.io with it. |
This lets jsonrpc build on stable (1.15) without needing serde_codegen