-
Notifications
You must be signed in to change notification settings - Fork 19
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
refactor: use FileLoader to load schedule files #7
Conversation
egg-core 需要改下,用 realpath 解析 plugin 目录 |
@@ -1,3 +1,3 @@ | |||
'use strict'; | |||
|
|||
exports.logrotater = true; | |||
exports.logrotator = 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.
这个插件是不是不用开了?下面那个用例自己造一个node_modules目录写个假任务。
不然耦合另一个插件来测试在这里不是很必要吧。
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.
原来就写着,问问 @dead-horse
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.
应该是可以干掉的, 后面再提个 PR 吧, 这个先合
+1 |
等 egg-core 合并重跑下 |
e217c3f
to
0dc6f1a
Compare
Current coverage is 80.70% (diff: 100%)@@ master #7 diff @@
==========================================
Files 3 3
Lines 119 114 -5
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
- Hits 98 92 -6
- Misses 21 22 +1
Partials 0 0
|
这个可以重新跑下了 |
已经重新跑了 |
2.1.0 |
+1 |
太棒!有了通用loader复用不再是问题 |
Checklist
npm test
passesAffected core subsystem(s)
Loader
Description of change
重写 Loader,简化代码
eggjs/egg#48