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

Fix #1691 - Add tuple deserialization support #1693

Merged
merged 2 commits into from Jun 29, 2017

Conversation

Projects
None yet
2 participants
@DBankov
Contributor

DBankov commented Feb 25, 2017

This PR adds tuple deserialization support. It also includes few fixes for the places where using tuple was not supported.

@DBankov DBankov changed the title from Fix for https://github.com/rejectedsoftware/vibe.d/issues/1691 - Added tuple deserialization support to Fix for #1691 - Added tuple deserialization support Feb 25, 2017

@DBankov DBankov changed the title from Fix for #1691 - Added tuple deserialization support to Fix #1691 - Added tuple deserialization support Feb 25, 2017

@DBankov DBankov changed the title from Fix #1691 - Added tuple deserialization support to Fix #1691 - Add tuple deserialization support Feb 25, 2017

@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Mar 10, 2017

Member

Looks good! I'll wait with the merge until after the 0.8.0 release, though, to avoid adding further weight (and potential breakage) to it.

Member

s-ludwig commented Mar 10, 2017

Looks good! I'll wait with the merge until after the 0.8.0 release, though, to avoid adding further weight (and potential breakage) to it.

@DBankov

This comment has been minimized.

Show comment
Hide comment
@DBankov

DBankov Mar 12, 2017

Contributor

Sounds great! Thanks for taking a look at this!

Contributor

DBankov commented Mar 12, 2017

Sounds great! Thanks for taking a look at this!

Dentcho Bankov added some commits Feb 25, 2017

@s-ludwig s-ludwig merged commit 01a41b7 into vibe-d:master Jun 29, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment