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

RFC: handle a deprecation warning from Cython (The keyword 'nogil' should appear at the end of the function signature line.) #4375

Merged
merged 1 commit into from
Mar 16, 2023

Conversation

neutrinoceros
Copy link
Member

@neutrinoceros neutrinoceros commented Mar 15, 2023

PR Summary

Fix a deprecation warning from Cython 3.0.0b1

The keyword 'nogil' should appear at the end of the function signature line. Placing it before 'except' or 'noexcept' will be disallowed in a future version of Cython

reduce the compatibility gap with Cython 3.1
This is a noop on Cython 0.29.x

…ould appear at the end of the function signature line.)
@neutrinoceros neutrinoceros added the refactor improve readability, maintainability, modularity label Mar 15, 2023
@neutrinoceros neutrinoceros marked this pull request as ready for review March 15, 2023 11:53
@neutrinoceros
Copy link
Member Author

@matthewturk does this look okay to you ?

@matthewturk matthewturk merged commit 7ef07cf into yt-project:main Mar 16, 2023
@neutrinoceros neutrinoceros deleted the cython3_nogil_order branch March 16, 2023 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor improve readability, maintainability, modularity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants