-
Notifications
You must be signed in to change notification settings - Fork 6
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
Modernize to latest rustc #5
Conversation
@@ -38,4 +40,4 @@ version = "*" | |||
optional = true | |||
|
|||
[dev-dependencies.deuterium_plugin] | |||
version = "*" | |||
path = "deuterium-plugin" |
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.
Please revert this because path deps can't be uploaded to the crates.io
@SkylerLipthay awesome work. Really awesome! Deuterium was published as v0.4.0. Please revert the path change in deuterium-plugin section and I merge this. |
Previous to this fix: if `delete` runs before `select` or `update`, `select` or `update` will fail.
Forced a push to remove the offending commit regarding the |
This chore work was a bit more involved than that of
deuterium-orm/deuterium
. Package should build and all tests should pass as soon as the newest changes todeuterium-orm/deuterium
are published to crates.io (I was developing this PR with a locally modifiedCargo.toml
).enum_primitive
crate. Hopefully a native solution comes around.syntax::print::pprust::ty_to_string
to replacesyntax::ext::quote::rt::ToSource
. I'm not quite sure ifpprust
is a one-to-one replacement, but it seems to work. Compiler plugins are always bound to break, anyway.