Conversation
Use features=['serde'] instead
Cargo.toml
Outdated
libc = "0.2" | ||
serde = {version = "1.0", optional = true} | ||
serde_derive = {version = "1.0", optional = true} | ||
serde = {version = "1.0", features=["derive"], optional = true} | ||
servo-skia = "0.30000008" |
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.
r+ with 0.30000009 here (yet to be published).
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.
Well it's not bumping euclid either. I can include it in this PR if you want to
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.
Oh right, sorry. Renaming the feature is a breaking change and the skia update is close, so maybe let’s land everything together?
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.
Sure
@bors-servo r+ Thanks! |
📌 Commit de0b8cd has been approved by |
Cleanup dependencies <!-- Reviewable:start --> This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-azure/281) <!-- Reviewable:end -->
☀️ Test successful - status-travis |
This change is