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

Improve serialization of rects and side offsets #327

Merged
merged 1 commit into from Jul 2, 2019
Merged

Conversation

@kvark
Copy link
Member

kvark commented Mar 14, 2019

Note: breaking change


This change is Reviewable

@kvark kvark requested a review from nical Mar 14, 2019
@nical
nical approved these changes Mar 14, 2019
@kvark kvark force-pushed the kvark:serde branch from ae924cf to ec4594a Mar 14, 2019
@kvark kvark changed the title Improve serialization of rects and sized offsets Improve serialization of rects Mar 14, 2019
@kvark
Copy link
Member Author

kvark commented Mar 14, 2019

Had to avoid touching the sized offsets because they use the proc macro magic for derives...
PTAL

@bors-servo
Copy link
Contributor

bors-servo commented Jul 2, 2019

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

@kvark kvark force-pushed the kvark:serde branch from ec4594a to e5a4802 Jul 2, 2019
@kvark kvark changed the title Improve serialization of rects Improve serialization of rects and side offsets Jul 2, 2019
@kvark
Copy link
Member Author

kvark commented Jul 2, 2019

@nical rebased now, PTAL

@nical
Copy link
Collaborator

nical commented Jul 2, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Jul 2, 2019

📌 Commit e5a4802 has been approved by nical

@bors-servo
Copy link
Contributor

bors-servo commented Jul 2, 2019

Testing commit e5a4802 with merge 1b78465...

bors-servo added a commit that referenced this pull request Jul 2, 2019
Improve serialization of rects and side offsets

Note: breaking change

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/euclid/327)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jul 2, 2019

☀️ Test successful - checks-travis
Approved by: nical
Pushing 1b78465 to master...

@bors-servo bors-servo merged commit e5a4802 into servo:master Jul 2, 2019
2 checks passed
2 checks passed
Travis CI - Pull Request Build Passed
Details
homu Test successful
Details
@kvark kvark deleted the kvark:serde branch Jul 2, 2019
bors-servo added a commit that referenced this pull request Jul 3, 2019
Automatically derive serde traits for more types

Similar to #327, adding rotations, translations and scales to the list of types deriving their serde implementations automatically.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/euclid/355)
<!-- Reviewable:end -->
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

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