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

Allow charset normalizer >=2 and <3 #6169

Merged
merged 1 commit into from
Jun 29, 2022

Conversation

frenzymadness
Copy link
Contributor

The current definition charset_normalizer~=2.0.0 does not allow charset-normalizer 2.1.0 which has been released a couple of hours ago. The newer version seems to be compatible with requests and the compatibility check suggests that requests should allow versions up to 3.0.

@frenzymadness
Copy link
Contributor Author

All tox environments passed locally.

@hswong3i
Copy link
Contributor

Similar and duplicated fix from #6175 (comment):

In between I give a quick dirty workaround for my packaging scripts, and it is now working with CentOS 8 + molecule 4.0.0 + charset_normalizer 2.1.0 + requests 2.28.0 (patched):

setup.cfg Outdated Show resolved Hide resolved
setup.py Outdated Show resolved Hide resolved
Copy link

@areche areche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change conforms with the compatibility checks made in requests/__init__.py

Copy link
Member

@nateprewitt nateprewitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks @frenzymadness! We'll release this in the next patch release, 2.28.1.

@nateprewitt nateprewitt merged commit 0e28c8f into psf:main Jun 29, 2022
@nateprewitt nateprewitt mentioned this pull request Jun 29, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants