Skip to content

don't call get_event_loop() if it's deprecated, handle RuntimeError from get_event_loop after asyncio.run#5799

Merged
willmcgugan merged 8 commits into
Textualize:mainfrom
graingert:fix-get-event-loop-call
May 31, 2025
Merged

don't call get_event_loop() if it's deprecated, handle RuntimeError from get_event_loop after asyncio.run#5799
willmcgugan merged 8 commits into
Textualize:mainfrom
graingert:fix-get-event-loop-call

Conversation

@graingert

@graingert graingert commented May 13, 2025

Copy link
Copy Markdown
Contributor

Fixes #5797 and #5788

Please review the following checklist.

  • Docstrings on all new or modified functions / classes
  • Updated documentation
  • Updated CHANGELOG.md (where appropriate)

@graingert graingert marked this pull request as ready for review May 13, 2025 10:47
@graingert

Copy link
Copy Markdown
Contributor Author

@willmcgugan this is ready for review now, sorry about the lint error

@willmcgugan

Copy link
Copy Markdown
Member

Hi @graingert There's an issue with Py3.8 Mind having a look?

@graingert

Copy link
Copy Markdown
Contributor Author

yep, just on it

@graingert

Copy link
Copy Markdown
Contributor Author

@willmcgugan fixed!

Comment thread src/textual/_compat.py
@graingert graingert requested a review from willmcgugan May 31, 2025 14:40
@willmcgugan

Copy link
Copy Markdown
Member

Thanks

@willmcgugan willmcgugan merged commit 185ba76 into Textualize:main May 31, 2025
23 checks passed
@graingert graingert deleted the fix-get-event-loop-call branch May 31, 2025 16:44
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.

Deprecation in Python 3.14 will break python -m textual and App.run()

2 participants