-
Notifications
You must be signed in to change notification settings - Fork 93
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: plugin support optionalDependencies #40
Conversation
@popomore, thanks for your PR! By analyzing the history of the files in this pull request, we identified @gxcsoccer to be a potential reviewer. |
3c9d2f8
to
50156c0
Compare
Current coverage is 99.41% (diff: 100%)@@ master #40 diff @@
==========================================
Files 16 16
Lines 669 681 +12
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
+ Hits 665 677 +12
Misses 4 4
Partials 0 0
|
function depCompatible(plugin) { | ||
if (plugin.dep && !(Array.isArray(plugin.dependencies) && plugin.dependencies.length)) { | ||
plugin.dependencies = plugin.dep; | ||
delete plugin.dep; |
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.
需要 depd 提示一下么?
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.
LGTM
不加了吧,文档不要出现就好了
fengmk2 <notifications@github.com>于2017年1月17日 周二00:30写道:
… ***@***.**** commented on this pull request.
------------------------------
In lib/loader/mixin/plugin.js <#40>:
> @@ -355,3 +361,10 @@ module.exports = {
},
};
+
+function depCompatible(plugin) {
+ if (plugin.dep && !(Array.isArray(plugin.dependencies) && plugin.dependencies.length)) {
+ plugin.dependencies = plugin.dep;
+ delete plugin.dep;
需要 depd 提示一下么?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#40>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAWA1d8eaOKjAX5prxoB_A-lyMRKEBi5ks5rS5sKgaJpZM4Lkn4P>
.
|
1.5.0 |
Checklist
npm test
passesAffected core subsystem(s)
loadPlugin
Description of change
Closes eggjs/egg#176