-
Notifications
You must be signed in to change notification settings - Fork 773
[feature]: add generate doc cmd for supernode #1100
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1100 +/- ##
==========================================
- Coverage 47.28% 47.25% -0.03%
==========================================
Files 118 116 -2
Lines 7244 7203 -41
==========================================
- Hits 3425 3404 -21
+ Misses 3549 3532 -17
+ Partials 270 267 -3
Continue to review full report at Codecov.
|
cmd/supernode/app/gen_doc.go
Outdated
@@ -36,7 +36,7 @@ func init() { | |||
genDocCommand := &GenDocCommand{} | |||
genDocCommand.cmd = &cobra.Command{ | |||
Use: "gen-doc", | |||
Short: "Generate Document for dfget command line tool with MarkDown format", | |||
Short: "Generate Document for supernode command line tool with MarkDown 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.
with MarkDown format
I think with
isn't appropriate,in
can be better?
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.
Agree
pkg/cmd/gen_doc.go
Outdated
genDocCommand := &GenDocCommand{} | ||
genDocCommand.cmd = &cobra.Command{ | ||
Use: "gen-doc", | ||
Short: "Generate Document for dfdaemon command line tool with MarkDown format", | ||
Short: fmt.Sprintf("Generate Document for %s command line tool with MarkDown format", name), |
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.
ditto
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
Signed-off-by: Starnop <[email protected]>
d32e824
to
c7ae59b
Compare
[feature]: add generate doc cmd for supernode
Signed-off-by: Starnop [email protected]
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
fixes #1097
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗?)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews