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.

Sign up for GitHub

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

Jump to bottom

WP-12916 handle encoding detector returning confidence of None #40

Merged
merged 3 commits into from
Nov 14, 2022
Merged

WP-12916 handle encoding detector returning confidence of None #40

merged 3 commits into from
Nov 14, 2022

Conversation

ChrisLing1
Copy link

@ChrisLing1 ChrisLing1 commented Nov 9, 2022 •
edited
Loading

https://varicent.atlassian.net/browse/WP-12916

Default to utf-8 when something goes wrong with encoding detection

@ChrisLing1 ChrisLing1 requested a review from jayma91 November 9, 2022 21:09
jayma91
jayma91 previously approved these changes Nov 9, 2022
View reviewed changes
@ChrisLing1 ChrisLing1 requested a review from jayma91 November 14, 2022 17:50
@ChrisLing1 ChrisLing1 merged commit d14345b into master Nov 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jayma91 jayma91 jayma91 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ChrisLing1 @jayma91