-
-
Notifications
You must be signed in to change notification settings - Fork 975
refactor(main.py): 使用anyio兼容层(默认后端为asyncio) 提高兼容性和可拓展性 #3272
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
你好 - 我已经审查了你的更改,它们看起来很棒!
帮助我变得更有用!请在每条评论上点击 👍 或 👎,我将利用这些反馈来改进你的评论。
Original comment in English
Hey there - I've reviewed your changes and they look great!
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
还有很多位置没改
anyio 没有 asyncio的队列,而是使用anyio.streams.memory 进行代替。 BREAKING CHANGE: 可能具有破坏性,稍后我会进行测试
|
@copilot 代码格式化 |
|
@LIghtJUNction I've opened a new pull request, #3274, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@copilot mypy工具进行检查,看看有没有错误,修复一下 |
|
@LIghtJUNction I've opened a new pull request, #3284, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@copilot 请你使用mypy进行检查,进修复异步相关的错误 |
|
@LIghtJUNction I've opened a new pull request, #3288, to work on those changes. Once the pull request is ready, I'll request review from you. |
fixes #XYZ
Motivation / 动机
动机:https://anyio.readthedocs.io/en/stable/why.html
Modifications / 改动点
Verification Steps / 验证步骤
Screenshots or Test Results / 运行截图或测试结果
Compatibility & Breaking Changes / 兼容性与破坏性变更
Checklist / 检查清单
requirements.txt和pyproject.toml文件相应位置。/ I have ensured that no new dependencies are introduced, OR if new dependencies are introduced, they have been added to the appropriate locations inrequirements.txtandpyproject.toml.Sourcery 总结
改进:
anyio.run替代asyncio.run,以支持多个异步后端并提高兼容性和可扩展性Original summary in English
Summary by Sourcery
Enhancements: