-
Notifications
You must be signed in to change notification settings - Fork 2
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
Gracefully handle invalid CSV files #118
Conversation
This LGTM but I want an approval from @steveoh on my last commit. Then I'm happy to fixup my commit and merge. |
I'll look at this Monday but will you fix the commit message |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's review this together when you get a chance.
a4d810b
to
631368d
Compare
closes #117 Co-Authored-By: steveoh <sgourley@utah.gov>
Includes a refactor to save us a full table scan.
631368d
to
0b2cba5
Compare
Thanks for contributing
Identify the issue referencing the feature or bug
#117
Description of the Change
Catch the errors and show specific instructions for resolving the problem.