Merged
Conversation
SunnyHaze
requested changes
Nov 13, 2025
Contributor
SunnyHaze
left a comment
SunnyHaze
left a comment
There was a problem hiding this comment.
Sorry，麻烦按照群里的文档再迭代一波，这个更好的做了try except和返回值的包装。感谢！
Member
Author
|
New requests were done. Please review, TQ |
SunnyHaze
requested changes
Nov 14, 2025
Contributor
SunnyHaze
left a comment
SunnyHaze
left a comment
There was a problem hiding this comment.
核心业务逻辑目测很好了！不过就是测试文件可能统一做成pytest做自动测试？这样会方便后续使用github自测。感谢！
Contributor
There was a problem hiding this comment.
这个Tester文件能不能写成pytest的格式作为单元测试，放在tests文件下面，通过pytest指令运行过测试即可。
SunnyHaze
approved these changes
Nov 14, 2025
Contributor
SunnyHaze
left a comment
SunnyHaze
left a comment
There was a problem hiding this comment.
LGTM, Sincerly thanks!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
✅ 任务状态管理 (5种状态)
✅ 任务信息追踪
✅ REST API (10个接口)
✅ 数据持久化