-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Feature/pgsql add pgsql unit test #1853
Conversation
Codecov Report
@@ Coverage Diff @@
## feature/pgsql #1853 +/- ##
=================================================
- Coverage 71.20% 71.18% -0.02%
=================================================
Files 463 463
Lines 43534 43534
=================================================
- Hits 30997 30990 -7
- Misses 10561 10568 +7
Partials 1976 1976
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
"fmt" | ||
|
||
_ "github.com/gogf/gf/contrib/drivers/pgsql/v2" | ||
_ "github.com/lib/pq" |
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.
github.com/lib/pq
在github.com/gogf/gf/contrib/drivers/pgsql/v2
中已经引用,这里不需要再显式引用。
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.
done
contrib/drivers/pgsql/pgsql.go
Outdated
"user=%s password=%s host=%s port=%s dbname=%s sslmode=disable", | ||
config.User, config.Pass, config.Host, config.Port, config.Name, | ||
"user=%s password=%s host=%s port=%s sslmode=disable", | ||
config.User, config.Pass, config.Host, config.Port, |
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.
这里不能去掉数据库名称配置,会引起数据库名称配置失效。
如果不想连接特定数据库,尝试设置数据库名称为空试试。
No description provided.