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

Wrap type conversion errors that occur while parsing in csv.ParseErrors #3

Merged
merged 2 commits into from
Dec 28, 2015

Conversation

jamesgarfield
Copy link

This provides richer information to the consumer, allowing them to know where int eh cvs the error occurred.

This provides richer information to the consumer, allowing them to know where int eh cvs the error occurred.
@ashultz0
Copy link

👍 one number question that could go either way

@quintenpalmer
Copy link

avoid the err = err; return err and 👍 from me!

@quintenpalmer
Copy link

🚢 it!

jamesgarfield added a commit that referenced this pull request Dec 28, 2015
Wrap type conversion errors that occur while parsing in csv.ParseErrors
@jamesgarfield jamesgarfield merged commit 10ff80f into master Dec 28, 2015
siddharth178 pushed a commit that referenced this pull request Mar 17, 2017
Allows the consumer to specify to leave out the header line in the csv that is generated as brought up in issue #3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants