Skip to content
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

melobot v3 tests for utilities levels #5

Merged
merged 11 commits into from
Dec 18, 2024
Merged

melobot v3 tests for utilities levels #5

merged 11 commits into from
Dec 18, 2024

Conversation

NyaCl
Copy link
Collaborator

@NyaCl NyaCl commented Sep 11, 2024

Basically some tests on melobot framework, including the basic utilities levels.

Edited 12.19.2024:
After discussion, we've considered to split multiple levels testing into individual PRs.

This project is definitely a lesson of async for me, since I NEVER tried those async tricks.
I just kept using threading stuffs (or sometimes multi-processing) to try to fasten my runtime efficiency.

I would always be grateful to have the chance communicating with dalao like u,
since we met in the group of Dr.Black XD

@NyaCl NyaCl requested review from aicorein and pk5ls20 September 11, 2024 19:12
@NyaCl NyaCl self-assigned this Sep 11, 2024
@NyaCl NyaCl removed the request for review from pk5ls20 September 11, 2024 19:14
@NyaCl NyaCl changed the title melobot tests for lower levels melobot v3 tests for lower levels Sep 25, 2024
@NyaCl
Copy link
Collaborator Author

NyaCl commented Sep 25, 2024

枕么还传了个 merge commit …… 到时候可能还需要 rebase,,

@aicorein
Copy link
Member

枕么还传了个 merge commit …… 到时候可能还需要 rebase,,

没事,最后写完统一压缩就行。

@aicorein aicorein marked this pull request as ready for review September 26, 2024 05:55
@aicorein
Copy link
Member

枕么还传了个 merge commit …… 到时候可能还需要 rebase,,

没事,最后写完统一压缩就行。

当然你也可以不压缩,这边合并之后会自动压缩。

@aicorein aicorein marked this pull request as draft September 26, 2024 05:58
NyaCl added 4 commits November 2, 2024 03:27
- Sequential await makes it not really 'async' (at least in py);
- Thanks for reviewing.
- lazy to change everywhere, just use it.
- @to_async still shows errors,
but since onebot tests were bare decls,
difficult to hint one by one,
let's just ignore them =.=
@NyaCl
Copy link
Collaborator Author

NyaCl commented Dec 17, 2024

传奇鸽王 ChlorideP 又整烂活辣(

然后不出所料又炸测试了。

姑且可以往下写了。

- just Unittest Orient Programming.
@aicorein
Copy link
Member

传奇鸽王 ChlorideP 又整烂活辣(

然后不出所料又炸测试了。

姑且可以往下写了。

加油

@aicorein
Copy link
Member

melobot.utils 部分的测试我抽空全部补好了。先合并这部分吧,剩下的测试干脆分成多个 PR。
感谢氯喵的付出哦,辛苦啦❤️

@aicorein aicorein marked this pull request as ready for review December 18, 2024 11:42
@aicorein aicorein merged commit 182dda6 into Meloland:main Dec 18, 2024
13 checks passed
@NyaCl
Copy link
Collaborator Author

NyaCl commented Dec 18, 2024

melobot.utils 部分的测试我抽空全部补好了。先合并这部分吧,剩下的测试干脆分成多个 PR。 感谢氯喵的付出哦,辛苦啦❤️

(蹭)

@NyaCl NyaCl changed the title melobot v3 tests for lower levels melobot v3 tests for utilities levels Dec 18, 2024
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