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

Ensure chardet is installed when testing using tox #548

Merged
merged 1 commit into from
Aug 13, 2021

Conversation

cjmayo
Copy link
Contributor

@cjmayo cjmayo commented Jul 27, 2021

Beautiful Soup uses chardet, if installed, to detect character
encodings. This can lead to different test results based on whether
chardet is installed or not.

Requests < 2.26.0 requires chardet, but since 2.26.0 Requests requires
charset_normalizer.

Explicitly installing chardet maintains consistent test results.

====

Testing for the first time here. Let's see if it closes #547.

Beautiful Soup uses chardet, if installed, to detect character
encodings. This can lead to different test results based on whether
chardet is installed or not.

Requests < 2.26.0 requires chardet, but since 2.26.0 Requests requires
charset_normalizer.

Explicitly installing chardet maintains consistent test results.
@cjmayo cjmayo merged commit b4d68ba into linkchecker:master Aug 13, 2021
github-actions bot pushed a commit that referenced this pull request Aug 13, 2021
Ensure chardet is installed when testing using tox b4d68ba
github-actions bot pushed a commit to cjmayo/linkchecker that referenced this pull request Aug 13, 2021
Ensure chardet is installed when testing using tox b4d68ba
@cjmayo cjmayo deleted the chardet branch October 18, 2021 18:40
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.

Test failures on master
1 participant