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

Make chardet/charset_normalizer optional dependencies #5875

Closed
wants to merge 4 commits into from
Closed

Make chardet/charset_normalizer optional dependencies #5875

wants to merge 4 commits into from

Commits on Jul 15, 2021

  1. Move chardet/charset_normalizer imports to .compat and remove version… 

    … checks
    @akx
    akx committed Jul 15, 2021
    Configuration menu
    Copy the full SHA
    0712b82 View commit details
    Browse the repository at this point in the history

  2. Make chardet/charset_normalizer an optional dependency 

    See psf#5871
    @akx
    akx committed Jul 15, 2021
    Configuration menu
    Copy the full SHA
    cf3fbf0 View commit details
    Browse the repository at this point in the history

  3. Make chardet/charset_normalizer extras, not required dependencies

    @akx
    akx committed Jul 15, 2021
    Configuration menu
    Copy the full SHA
    03ee569 View commit details
    Browse the repository at this point in the history

  4. Add tests to check against false positives for more esoteric encodings

    @akx
    akx committed Jul 15, 2021
    Configuration menu
    Copy the full SHA
    2d46ddb View commit details
    Browse the repository at this point in the history