Skip to content

Conversation

@cpcloud
Copy link
Contributor

@cpcloud cpcloud commented Nov 9, 2017

No description provided.

@wesm
Copy link
Member

wesm commented Nov 10, 2017

May need to use BytesIO in the test case

@cpcloud
Copy link
Contributor Author

cpcloud commented Nov 10, 2017

This needs at least one additional test for columns with named indexes. I'll crank this out early tomorrow.

@cpcloud
Copy link
Contributor Author

cpcloud commented Nov 11, 2017

@wesm This is ready for review when you have a chance.

Copy link
Member

@wesm wesm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, thanks @cpcloud!

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