MoChengqian commented on PR #389:
URL:
https://github.com/apache/dubbo-go-hessian2/pull/389#issuecomment-4141238541
上次commit[6965a4e]CI失败原因(已修复):
1.没有把 golangci-lint 全量跑透
2.局部验证,颗粒度不够细
老债:
1. 老代码风格问题。
像 array_test.go:131 这种显式写 float64 类型，放在以前没事，但现在 staticcheck 会报
QF1011，认为这种类型可以省略。
2. 老实现用了已弃用 API。
string.go:41 原来用的是 reflect.SliceHeader 和 reflect.StringHeader 做零拷贝转换；Go
1.21 之后这套写法被标记为 deprecated，所以 CI 现在会报 SA1019。
总结:owner
意识不够强，不够端到端,不能因为只是改CI错误、改动小,就只做局部验证,不在本地把测试跑满.以后定会更加细致严谨,在每一次提交之前都做好本地闭环验证,并检查新增代码的规范性问题和可维护性问题,减少返工😁
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
