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

removed dependency on serde #75

Closed

Conversation

@jbg
Copy link

jbg commented Feb 17, 2017

The Serialize/Deserialize implementations have moved to webrender.

See servo/servo#15607

Other related pull requests:


This change is Reviewable

@nox
Copy link
Member

nox commented Feb 17, 2017

This will have to wait for serde 0.9 to land first, am not changing the serde bump now.

@nox nox changed the title removed dependency on serde (Do not merge) removed dependency on serde Feb 17, 2017
@bors-servo
Copy link
Contributor

bors-servo commented Feb 17, 2017

The latest upstream changes (presumably #69) made this pull request unmergeable. Please resolve the merge conflicts.

@nox nox changed the title (Do not merge) removed dependency on serde removed dependency on serde Feb 18, 2017
@nox
Copy link
Member

nox commented Feb 18, 2017

Serde 0.9 landed, this is now unblocked!

@nox nox self-assigned this Mar 3, 2017
@jdm
Copy link
Member

jdm commented Mar 20, 2017

@jbg Can you rebase this against master?

@KiChjang
Copy link
Member

KiChjang commented Apr 5, 2017

Closing due to inactivity.

@KiChjang KiChjang closed this Apr 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.