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

Upgrade Cython for Python 3.11 #80

Closed
wants to merge 2 commits into from
Closed

Conversation

cclauss
Copy link

@cclauss cclauss commented Jul 19, 2022

A minimal subset of #78

An on-demand GitHub Action:

name: pip_install_cchardet
on: [pull_request, push, workflow_dispatch]
jobs:
  pip_install_cchardet:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/setup-python@v4
        with:
          python-version: 3.11
      - run: python3.11 -m pip install --upgrade cython>=0.29.30
      - run: python3.11 -m pip install cchardet  # Will succeed only if cython>=0.29.30

@jameshilliard
Copy link

@PyYoshi Think we could get a new release with this updated as python 3.11 is now released?

@cclauss cclauss changed the title Upgrade Cython for Python 3.11-beta.4 Upgrade Cython for Python 3.11 Oct 29, 2022
@SimplicityGuy
Copy link

Without this fix, this is preventing many downstream packages from being operational. @PyYoshi, please take a look and create a new release to include support for Python 3.11.

@@ -7,7 +7,7 @@ verify_ssl = true
chardet = "==3.0.4"
nose = "==1.3.7"
tox = "==3.20.1"
Cython = "==0.29.21"
Cython = "==0.29.30"

Choose a reason for hiding this comment

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

Latest version is 0.29.32. Consider upgrading to that version.

@cclauss
Copy link
Author

cclauss commented Nov 6, 2022

Closing in favor of #78 which really should be merged.

@cclauss cclauss closed this Nov 6, 2022
@cclauss cclauss deleted the patch-2 branch November 6, 2022 09:17
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.

3 participants