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: can disable fast ready #4
Conversation
@fengmk2, thanks for your PR! By analyzing the history of the files in this pull request, we identified @jtyjty99999 and @atian25 to be potential reviewers. |
exports.development = { | ||
watchDirs: [], // dirs needed watch, when files under these change, application will reload. | ||
ignoreDirs: [], // dirs don't need watch, including subdirectories | ||
fastReady: 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.
missing comment
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.
嗯, 看到了
@@ -4,8 +4,10 @@ | |||
* @member Config#development | |||
* @property {Array} watchDirs - dirs needed watch, when files under these change, application will reload, use relative path | |||
* @property {Array} ignoreDirs - dirs don't need watch, including subdirectories, use relative path | |||
* @property {Boolean} fastReady - don't wait all plugins ready, default is 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.
不需要,没人会设置的。
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.
那要在 readme 里面说明下默认开启且可能导致的错误场景? 内网版不是有人踩坑了么
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.
不加了,加了反而更加麻烦。内网也就一个人觉得会有问题。如果需要 fastready = false,直接关闭这个插件更好。。。
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.
+1
1ff9cdd
to
c4c543e
Compare
Current coverage is 100% (diff: 100%)@@ master #4 diff @@
===================================
Files 4 4
Lines 44 44
Methods 0 0
Messages 0 0
Branches 0 0
===================================
Hits 44 44
Misses 0 0
Partials 0 0
|
合了, 你发吧 |
1.1.0 |
Checklist
npm test
passesAffected core subsystem(s)
Description of change