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

Fix an error for `CSV.open` #131

Merged
merged 1 commit into from Jun 4, 2020
Merged

Fix an error for `CSV.open` #131

merged 1 commit into from Jun 4, 2020

Conversation

@koic
Copy link
Contributor

@koic koic commented Jun 4, 2020

Follow up to https://github.com/ruby/csv/pull/130/files#r434885191.

This PR fixes ArgumentError for CSV.open when processing invalid byte sequence in UTF-8.

Follow up to https://github.com/ruby/csv/pull/130/files#r434885191.

This PR fixes `ArgumentError` for `CSV.open` when processing
invalid byte sequence in UTF-8.
@kou
Copy link
Member

@kou kou commented Jun 4, 2020

Thanks.

It seems that our benchmark doesn't show significant slow down:

Before: https://github.com/ruby/csv/runs/729090943?check_suite_focus=true
After: https://github.com/ruby/csv/pull/131/checks?check_run_id=736633688

(I want to compare benchmark results automatically...)

@kou kou merged commit a4b528c into ruby:master Jun 4, 2020
17 checks passed
17 checks passed
Benchmark: Ruby 2.7: macos-latest
Details
Normal test: Ruby 2.5: macos-latest
Details
Benchmark: Ruby 2.7: ubuntu-latest
Details
Normal test: Ruby 2.5: ubuntu-latest
Details
Benchmark: Ruby 2.7: windows-latest
Details
Normal test: Ruby 2.5: windows-latest
Details
Normal test: Ruby 2.6: macos-latest
Details
Normal test: Ruby 2.6: ubuntu-latest
Details
Normal test: Ruby 2.6: windows-latest
Details
Normal test: Ruby 2.7: macos-latest
Details
Normal test: Ruby 2.7: ubuntu-latest
Details
Normal test: Ruby 2.7: windows-latest
Details
Scanner test: Ruby 2.7: macos-latest
Details
Scanner test: Ruby 2.7: ubuntu-latest
Details
Scanner test: Ruby 2.7: windows-latest
Details
Gem test
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@koic koic deleted the koic:fix_an_error_for_csv_open branch Jun 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.