-
Notifications
You must be signed in to change notification settings - Fork 135
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
feat: customize load in agent or in app #88
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #88 +/- ##
==========================================
- Coverage 85.71% 84.61% -1.10%
==========================================
Files 4 4
Lines 63 65 +2
==========================================
+ Hits 54 55 +1
- Misses 9 10 +1
Continue to review full report at Codecov.
|
@@ -1,6 +1,8 @@ | |||
'use strict'; | |||
|
|||
exports.sequelize = { | |||
agent: true, |
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.
文档里面加一下这两个参数
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
@@ -82,6 +82,8 @@ egg-sequelize has a default sequelize options below | |||
|
|||
```js | |||
{ | |||
app: true, |
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.
解释一下这两个参数的含义
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
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.
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.
好,是我看错了
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.
💯
这个可以 merge 发包吗 |
这个可以发个包么。🙏 |
Checklist
npm test
passesAffected core subsystem(s)
Description of change
根据配置选择是在agent,还是app上初始化sequelize。