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

added .csv support #64

Merged
merged 4 commits into from Sep 5, 2014

Conversation

Projects
None yet
2 participants
@deanmalmgren
Owner

deanmalmgren commented Aug 29, 2014

To supplement the other spreadsheet support we have (.xls and .xlsx), I thought I'd throw this together really quickly. I was hoping to be able to automatically dectect which delimiters are used but that was failing miserably on the tests/csv/raw_text.csv test. If anyone has any insights into how to get that to work, I'd greatly appreciate it!

@coveralls

This comment has been minimized.

coveralls commented Aug 29, 2014

Coverage Status

Coverage increased (+0.52%) when pulling e7053b5 on csv into 28a4628 on master.

@coveralls

This comment has been minimized.

coveralls commented Aug 29, 2014

Coverage Status

Coverage increased (+0.52%) when pulling 0219f11 on csv into 28a4628 on master.

deanmalmgren added a commit that referenced this pull request Sep 5, 2014

@deanmalmgren deanmalmgren merged commit e538ba8 into master Sep 5, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

@deanmalmgren deanmalmgren deleted the csv branch Sep 5, 2014

deanmalmgren added a commit that referenced this pull request Sep 5, 2014

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