Quickly start developing curveadm based on cobra #337
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add basic instructions for using the cobra library and an introduction to curveadm CLI development Signed-off-by: Liao PengFei <[email protected]>
caoxianfei1
reviewed
Nov 14, 2023
cli/cli_cn.md
Outdated
| @@ -0,0 +1,131 @@ | |||
| # Curveadm CLI 开发 | |||
|
|
|||
| ## 命令行功能项目结构 | |||
There was a problem hiding this comment.
我觉得先介绍cobra库比较好，简单介绍一下。
重点介绍一下curveadm的程序的组织方式，比如入口mian函数，以及到cli的组织方式
Signed-off-by: Liao PengFei <[email protected]>
Signed-off-by: Liao PengFei <[email protected]>
Signed-off-by: Liao PengFei <[email protected]>
caoxianfei1
reviewed
Nov 20, 2023
| ... | ||
| } | ||
| ... | ||
| addSubCommands(cmd, curveadm) |
There was a problem hiding this comment.
这里可以更详细一点吗？举个例子更详细的描述一下，比如是如何嵌套子命令的，以及子命令最终形式是什么。
包括后面的internal目录下是怎么串接起来的，可以更细一点。
|
LGTM! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fix #265
Add basic instructions for using the cobra library and an introduction to curveadm CLI development