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

Separate Sexp and List in BinaryWriter #59

Merged

tgregg merged 1 commit into amazon-ion:master from R-maan:containers-in-writer

Jan 29, 2020

Contributor

R-maan commented Jan 29, 2020

Description of changes:

IonType.List and IonType.Sexp were combined as ContainerType.Sequence and returning the List code for both cases.


          Separate Sexp and List in BinaryWriter

d3e7237

tgregg approved these changes

View reviewed changes

tgregg merged commit f61c31e into amazon-ion:master

pbcornell commented Jan 29, 2020

Why didn't the tests in this repo identify this bug?

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

Labels

None yet