kuscia icon indicating copy to clipboard operation
kuscia copied to clipboard
verified publisher icon secretflow

为 Kuscia 添加必要的注释与单测

Open Ldmmx opened this issue 7 months ago • 4 comments

Fixed #521

Ldmmx avatar May 06 '25 06:05 Ldmmx

部分测试函数失败，需要修改cmd目录外的测试，因此，没有修改

Ldmmx avatar May 06 '25 06:05 Ldmmx

部分测试函数失败，需要修改cmd目录外的测试，因此，没有修改

cmd/example/datamesh/dm_server_test.go:23:25: cannot call pointer method Flags on cobra.Command (typecheck)

可以看下 github 的 ci 日志，有些单测写的不太对。

yuluo-yx avatar May 06 '25 07:05 yuluo-yx

可以根据 aci 中的 image 查看新增的单测是否能够跑通，确保 aci 能够正确执行成功

gaoyonglong avatar May 08 '25 07:05 gaoyonglong

你好，已按照要求修改，请审查

Ldmmx avatar May 12 '25 03:05 Ldmmx

已经完成： 1.不必要改的格式还原 2.新增文件空最后一行 3.许可证头部 4.常量替换

Ldmmx avatar May 12 '25 12:05 Ldmmx