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

Resolves PR80, prep for new driver release #82

Merged
merged 2 commits into from
Aug 4, 2016
Merged

Resolves PR80, prep for new driver release #82

merged 2 commits into from
Aug 4, 2016

Conversation

bchavez
Copy link
Owner

@bchavez bchavez commented Aug 4, 2016

Supersedes PR #80, all unit tests should now be passing.

Minor tweaks to previous #79 and #78.

Preps for new driver release.

💼 👔 _"Taking care of business every day... Taking care of business every way..."_

@bchavez bchavez merged commit 714a2de into master Aug 4, 2016
@bchavez bchavez deleted the issue_80 branch August 4, 2016 01:50
@oliverjanik
Copy link

Alright, this works for me.

Looks like I'll just need to modify the default JsonSettings.

@bchavez
Copy link
Owner Author

bchavez commented Aug 4, 2016

Published v2.3.10.

Alrighty, Imma head out for some jogging, be back in a few hours. Hope everything goes well =)

The Converter.Serializer custom settings holder will likely change with the upcoming #77 protocol performance speed-ups and consequently allow #33 a Jill Response deserializer soon.

Take care man, and thanks a bunch for bringing this to my attn.

Brian

♣️ 🎲 _"I'm just doing ma thang, you know the Units the game..."_

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants