Skip to content

Ensure chardet is installed when testing using tox#548

Merged
cjmayo merged 1 commit intolinkchecker:masterfrom
cjmayo:chardet
Aug 13, 2021
Merged

Ensure chardet is installed when testing using tox#548
cjmayo merged 1 commit intolinkchecker:masterfrom
cjmayo:chardet

Conversation

@cjmayo
Copy link
Copy Markdown
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