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

Allow replacement of bad encoding #73

Merged

Conversation

malcolmstill
Copy link
Contributor

@malcolmstill malcolmstill commented Nov 2, 2017

When processing CSV files we may encounter fields which are
poorly encoded. These fields may not be of concern to the
task at hand and such bad encoding can be ignored.

Herein an option :replacement is introduced, the value of which
is used to replace bad codepoints. It defaults to nil which disables
any replacement.

When processing CSV files we may encounter fields which are
poorly encoded. These fields may not be of concern to the
task at hand and such bad encoding can be ignored.

Herein an option `:replacer` is introduced, the value of which
is used to replace bad codepoints. It defaults to `nil` which disables
any replacement.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 98.601% when pulling 17c6bb5 on malcolmstill:bad-encoding-replacement into 99c2e46 on beatrichartz:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 98.601% when pulling 6ab70bb on malcolmstill:bad-encoding-replacement into 99c2e46 on beatrichartz:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 98.601% when pulling e32fc93 on malcolmstill:bad-encoding-replacement into 99c2e46 on beatrichartz:master.

@beatrichartz beatrichartz merged commit e936399 into beatrichartz:master Dec 20, 2017
@beatrichartz
Copy link
Owner

Thanks for this! Merged.

@beatrichartz
Copy link
Owner

Released in 2.1.0

@malcolmstill malcolmstill deleted the bad-encoding-replacement branch August 22, 2019 10:44
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.

None yet

3 participants