-
Notifications
You must be signed in to change notification settings - Fork 230
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
fix: handle exception cases while no plugins #142
Conversation
salomezhang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
|
||
console.log('You can search more templates or plugins through https://feflowjs.com/encology/'); | ||
console.log('==============================================='); | ||
if (!list.length) { | ||
console.log(chalk.magenta('暂未安装任何模板和插件')); |
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.
这个地方统一改成英文
} | ||
}); | ||
if (!flag) { | ||
console.log(chalk.magenta('暂未安装任何插件')); |
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.
这个地方统一改成英文
} | ||
|
||
flag = 0; | ||
// print plugins list |
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.
这行注释意义不大可以删掉
@@ -32,21 +32,39 @@ function loadModuleList(ctx: any) { | |||
module.exports = (ctx: any) => { | |||
ctx.commander.register('list', 'Show all plugins installed.', () => { | |||
const list = loadModuleList(ctx); | |||
let flag = 0; |
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.
这个 flag 命名需要调整,可以改成 templateCnt 和 pluginCnt。不要一个 flag 用多次
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.
LGTM
No description provided.