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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix test errors caused by updated chardet v5 release #6777

Merged
merged 1 commit into from
Jun 29, 2022

Conversation

christian-monch
Copy link
Contributor

@christian-monch christian-monch commented Jun 29, 2022

Fixes #6776

This PR limits the allowed chardet version to <5.0.0, because requests does only support chardet up to version 4, and version 5.0.0. was released on pypi on 25th of June 2022

Changelog

馃彔 Internal

  • limit chardet version to version 4 in order to fulfill requests-dependencies

This commit specifies that a chardet version larger
than 3.0.4 and smaller than 5.0.0 is required. The
reason for the upper limit is that "requests" does
only support chardet version 4.
@christian-monch christian-monch added the semver-internal Changes only affect the internal API label Jun 29, 2022
@christian-monch christian-monch merged commit bac08c8 into datalad:maint Jun 29, 2022
@christian-monch
Copy link
Contributor Author

Merging this since it just changes a dependency definition and seems to fix the recently appearing test problem

@christian-monch
Copy link
Contributor Author

Cherry pick commit 0c88872 to your branch to get the fix without rebasing

@christian-monch christian-monch deleted the issue-6776-fix-tests branch June 29, 2022 13:32
@yarikoptic
Copy link
Member

This PR limits the allowed chardet version to <5.0.0, because requests does only support chardet up to version 4

was issue discovered/fixed/filed in requests on this?

@christian-monch
Copy link
Contributor Author

Not yet, will do that

@christian-monch
Copy link
Contributor Author

I did not find an existing issue in requests that discusses the situation.

Filed a new issue psf/requests#6183

@github-actions
Copy link

github-actions bot commented Jul 6, 2022

馃殌 PR was released in 0.16.7 馃殌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released semver-internal Changes only affect the internal API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants