Skip to content

enable the cython binding=True directive#484

Open
graingert wants to merge 4 commits intoMagicStack:masterfrom
graingert:enable-cython-binding-directive
Open

enable the cython binding=True directive#484
graingert wants to merge 4 commits intoMagicStack:masterfrom
graingert:enable-cython-binding-directive

Conversation

@graingert
Copy link
Copy Markdown
Contributor

it becomes the default in cython3 and is needed for sniffio
to detect uvloop as asyncio efficiently

it becomes the default in cython3 and is needed for sniffio
to detect uvloop as asyncio efficiently
Comment thread tests/test_base.py Outdated
Comment thread tests/test_base.py Outdated
@graingert
Copy link
Copy Markdown
Contributor Author

Hmm, it might be worth updating to cython 3 instead?

@fantix
Copy link
Copy Markdown
Member

fantix commented Sep 13, 2022

Oh let me try that.


OK, I'd probably do Cython 3 in another uvloop release, it'll take some time to adapt to Cython 3, also Cython 3 would be more tested at that time I think.

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.

2 participants