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.

Sign up for GitHub

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

Jump to bottom

Avoid a JsonException that gets thrown when there are ignored properties under some circumstances. #144

Open

airbreather wants to merge 4 commits into develop from gh143

Member

airbreather commented Apr 24, 2024

Fixes #143

The new tests fail when added to develop and pass on this branch.

airbreather added 4 commits

April 24, 2024 15:32


          This should fix it... needs a test or three, though.


          add some tests that pass with the fix and fail without the fix

c581d9b


          try to remove some whitespace-only changes

bbd3fa1


          add a comment explaining why we do this instead of Skip()

0a570c1

(IMO, Skip() really should have been implemented this way, but whatever)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet