Skip to content

Support Python 3.14#145

Merged
unkcpz merged 1 commit intoaiidateam:masterfrom
danielhollas:py3.14
Oct 22, 2025
Merged

Support Python 3.14#145
unkcpz merged 1 commit intoaiidateam:masterfrom
danielhollas:py3.14

Conversation

@danielhollas
Copy link
Contributor

@danielhollas danielhollas commented Oct 22, 2025

Only one trivial deprecation warning, otherwise things seems to just work.


def ensure_coroutine(coro_or_fn):
if asyncio.iscoroutinefunction(coro_or_fn):
if inspect.iscoroutinefunction(coro_or_fn):
Copy link
Contributor Author

Choose a reason for hiding this comment

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

asyncio.iscoroutinefunction was deprecated in 3.14,
inspect.iscoroutinefunction is in Python since 3.5 so safe to switch to
https://docs.python.org/3/library/inspect.html#inspect.iscoroutinefunction

@danielhollas danielhollas changed the title Test with Python 3.14 Support Python 3.14 Oct 22, 2025
@danielhollas danielhollas marked this pull request as ready for review October 22, 2025 13:03
@danielhollas
Copy link
Contributor Author

@unkcpz this one should be easy to review 🙏 👼

Copy link
Member

@unkcpz unkcpz left a comment

Choose a reason for hiding this comment

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

looks all good, thanks!

@danielhollas what is your opinion on drop py3.9 support?

@danielhollas
Copy link
Contributor Author

danielhollas commented Oct 22, 2025

I don't want to drop support since we're still using it in AiiDAlab. And honestly for this package which is very stable I don't see any advantage of dropping support, it would not give us anything.

@unkcpz
Copy link
Member

unkcpz commented Oct 22, 2025

see this aiidateam/aiida-core#6672 I didn't realize they drop the py3.9 support in the new release. The only thing I want is to take that one line fix I did 1 year ago. It can be a nice fix that people not suffer from verdi daemon stop timeout.

@unkcpz unkcpz merged commit 4a64857 into aiidateam:master Oct 22, 2025
9 of 10 checks passed
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