-
Notifications
You must be signed in to change notification settings - Fork 22
Conversation
@@ -105,7 +105,7 @@ fn find_definition_request_from_json() { | |||
assert_eq!(req.column, 3); | |||
} | |||
|
|||
#[derive(Debug, Deserialize, Serialize)] | |||
#[derive(Debug, Serialize, Deserialize)] |
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.
Is the order important? I don't think I would ever have thought of that!
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.
It doesn't actually matter. This was an attempt to fix a weird error that turned out to be due to an old version of serde_codegen. I'll revert this line.
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.
Ok, thanks for explaining!
Some dependencies don't guarantee beta compatibility making it difficult for racerd to do so. In this case, the latest version of clap (racer dep) is broken.
I filed contain-rs/vec-map#22 about the vec_map issue. |
Thanks! I wonder if that was a bug in beta? Current nightly doesn't seem to have that issue. |
Resolves #50, #48, #45.
This change is