Closed Ruomenger closed 11 months ago
@Ruomenger 2-5 都没问题。但我对第一条有不同看法。
所谓的 yoda conditions 只是 staticcheck 不建议使用。但 yoda conditions 本身没有问题。纯粹为了不让 staticcheck 显示警告就在 monkey_test.go 引入大量功能无关的改动,我感觉不太合适。
根据 staticcheck 的文档,我们可以在项目根目录创建配置文件staticcheck.conf
checks = ["all", "-ST1000", "-ST1003", "-ST1017"]
这样可以排除掉一些不是很契合的规则。
@Ruomenger 2-5 都没问题。但我对第一条有不同看法。
所谓的 yoda conditions 只是 staticcheck 不建议使用。但 yoda conditions 本身没有问题。纯粹为了不让 staticcheck 显示警告就在 monkey_test.go 引入大量功能无关的改动,我感觉不太合适。
根据 staticcheck 的文档,我们可以在项目根目录创建配置文件
staticcheck.conf
checks = ["all", "-ST1000", "-ST1003", "-ST1017"]
这样可以排除掉一些不是很契合的规则。
我明白增加这个配置修改是可以避免这个告警,只是这种写法在go
中非常奇怪,无论是在go
标准库还是大部分第三方库中都没有这种写法。这种写法一般是用在C/C++
中可以避免一些在==
误写为=
时的问题(因为隐式类型转换),但在go
中=
赋值就并非布尔值编译时就会直接报错,不会有这种错误问题。
修复项目中的一些警告问题,主要有:
@taoso FYI