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

Json.NET DateParseHandling set to None #80

Closed

Conversation

oliverjanik
Copy link

Failing test + extra setting on the default converter. CLA signed.

@bchavez
Copy link
Owner

bchavez commented Aug 3, 2016

Thanks @oliverjanik , going to take a second look at this. I'll let you know what I find out.

🎧 🎶 _Antiserum & Mayhem - Trippy_

@oliverjanik
Copy link
Author

Hmm wondering why the CI failed.

@bchavez
Copy link
Owner

bchavez commented Aug 4, 2016

I might have a working solution for you, but I'm getting some inconsistent Grouping tests that sometimes pass and sometimes fail, not totally sure why yet. Still feels like a mysteirous CLR thing at the moment.

💨 🚶 _"Bubbles of gas in my brain... Send me off balance, it's not enough"_

bchavez added a commit that referenced this pull request Aug 4, 2016
@bchavez
Copy link
Owner

bchavez commented Aug 4, 2016

Hey @oliverjanik, I think I got it. Waiting on CI to check PR #82.

We'll ship a new driver release tonight. Thanks for your patience.

Thanks,
Brian

🌹 🌷 The Chainsmokers - Roses (feat. ROZES)

@bchavez bchavez closed this Aug 4, 2016
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