Skip to content

Conversation

SAKURA-CAT
Copy link
Member

@SAKURA-CAT SAKURA-CAT commented Jun 19, 2024

Description

swanlab全面接入swankit,功能解藕,环境变量明确,测试脚本重构。
将媒体类型处理改为可选依赖 #633
更新了贡献文档 #634

有关swanlab-toolkit的文档请查看: https://github.com/SwanHubX/SwanLab-Toolkit/wiki ,时间关系还没完全写完

功能解藕

log、callback和部分共享环境变量存放在swankit包中,作为基础开发工具的一部分

环境变量明确

删除了原本十分麻烦的的环境变量记忆功能,仅保留部分有用的环境变量,现在所有的环境变量通过api动态计算

测试脚本重构

  • 明确测试流程,在每个脚本执行过后,恢复测试环境原状
  • 由于swanlab会产出日志文件,因此在测试完毕后,检查是否在不该产出的地方产出了日志文件
  • 重写一些单元测试函数

对于测试,删除了原本的config.json文件,并且增加了对.env环境变量的配置依赖。在项目根目录下创建.env文件,写入如下内容:

TEST_CLOUD_KEY=你的apikey

如果你不想进行云端部分业务的测试,可以填入:

TEST_CLOUD_SKIP=true

此时TEST_CLOUD_KEY 不再有用,参考CONTRIBUTING.md文件内描述。

此外,package.mock.json文件被移到了项目根目录下。

@SAKURA-CAT SAKURA-CAT self-assigned this Jun 19, 2024
@SAKURA-CAT SAKURA-CAT added 🐛 bug Something isn't working 💪 enhancement New feature or request good attempt labels Jun 19, 2024
@SAKURA-CAT SAKURA-CAT mentioned this pull request Jun 21, 2024
@SAKURA-CAT SAKURA-CAT marked this pull request as ready for review June 22, 2024 12:31
@ShaohonChen
Copy link
Contributor

contribute没啥问题,等 @Feudalman 看看?

@SAKURA-CAT SAKURA-CAT merged commit 921d438 into main Jun 23, 2024
@SAKURA-CAT SAKURA-CAT deleted the feat/toolkit branch June 23, 2024 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug Something isn't working 💪 enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants