-
Notifications
You must be signed in to change notification settings - Fork 5
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 error return but uncheck and fix bug #2
Conversation
@@ -22,7 +22,7 @@ package main | |||
|
|||
import ( | |||
"fmt" | |||
|
|||
"errors" |
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.
system packages should be grouped in first block
cmd/gyee/configcmd.go
Outdated
Name: "config", | ||
Usage: "Manage config", | ||
Category: "CONFIG COMMANDS", | ||
Description: ` |
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.
leading newline here was shown in help text, no need to format
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.
ok
I see u've got 7 commits with empty and merge along the way. |
master branch is protected by default, you may find a dedicated patch branch handy. |
OK |
you can close this pr |
修改saveConfig 函数返回值,由nil修改为具体的错误类型
修复SaveConfigToFile遇到错误应该直接返回error,而不是继续执行下面的代码