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

Fixes line content incorrectly being put into the row column field when there is an encoding error. #130

Merged

Conversation

glacier
Copy link

@glacier glacier commented Jul 10, 2015

No description provided.

@glacier
Copy link
Author

glacier commented Jul 10, 2015

Can someone review this? I'm hoping to ship this gem in my codebase and I'd really need to get this bug fixed. Thanks!

pezholio added a commit that referenced this pull request Jul 13, 2015
Fixes line content incorrectly being put into the row column field when there is an encoding error.
@pezholio pezholio merged commit ad38039 into Data-Liberation-Front:master Jul 13, 2015
@pezholio
Copy link
Contributor

Done. Thanks for the PR! 😄

@pezholio
Copy link
Contributor

I'll give you a shout when I've bumped the release in Rubygems

@pezholio
Copy link
Contributor

This is now live in Rubygems https://rubygems.org/gems/csvlint/versions/0.1.1

@glacier
Copy link
Author

glacier commented Jul 15, 2015

@pezholio thank you for merging! Thank you! Does this mean that if I get the updated gem now, it'll have the fix?

@pezholio
Copy link
Contributor

Yup 😄

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