-
Notifications
You must be signed in to change notification settings - Fork 157
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
新增了sqlx.NewNullXXX系列方法 #227
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #227 +/- ##
==========================================
+ Coverage 95.71% 95.84% +0.12%
==========================================
Files 56 57 +1
Lines 3082 3106 +24
==========================================
+ Hits 2950 2977 +27
+ Misses 101 99 -2
+ Partials 31 30 -1
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
修复一下下方的lint错误, 并提供单元测试 |
收到 |
补充了license的内容;
sqlx/newnull_test.go
Outdated
type args struct { | ||
val bool | ||
} | ||
tests := []struct { | ||
name string | ||
args args | ||
want sql.NullBool |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
把args结构体中的val内容平铺在tests := []struct {...},并去掉args结构体
检查下方所有测试,使其满足上述要求
sqlx/newnull_test.go
Outdated
want sql.NullBool | ||
}{ | ||
{ | ||
name: "test", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
命名不准确,还有重复,应该描述测试场景
检查下方所有测试,使其满足上述要求
学会查看lint提示, 点击下方 "Format Go code" 末尾的Details查看详细的报错信息并修复问题 |
#222