Skip to content

fix: replace asyncio.ensure_future with create_task#62

Open
0b0d3 wants to merge 1 commit intodvoraj75:mainfrom
0b0d3:fix/replace-ensure-future-with-create-task
Open

fix: replace asyncio.ensure_future with create_task#62
0b0d3 wants to merge 1 commit intodvoraj75:mainfrom
0b0d3:fix/replace-ensure-future-with-create-task

Conversation

@0b0d3
Copy link
Copy Markdown

@0b0d3 0b0d3 commented Mar 21, 2026

Summary

  • Replaced asyncio.ensure_future() with asyncio.create_task() in indicator/app.py and indicator/client.py
  • Updated documentation in docs/modules/indicator.md to reflect the change
  • ensure_future is deprecated; create_task is the recommended replacement

Closes #32

Test plan

  • Existing tests pass with create_task in place of ensure_future
  • Indicator app still schedules async coroutines correctly from sync callbacks
  • Client reconnection logic still works as expected

🤖 Generated with Claude Code

asyncio.ensure_future is deprecated in favor of asyncio.create_task.
Updated both indicator/app.py and indicator/client.py, plus the
corresponding documentation.

Closes dvoraj75#32

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

Replace asyncio.ensure_future with create_task

1 participant