-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
[Docs] Updating the docs for mesheryctl exp command #11204 #11225
[Docs] Updating the docs for mesheryctl exp command #11204 #11225
Conversation
01d82bb
to
1ed6286
Compare
🚀 Preview for commit 1ed6286 at: https://6674ccc4edf85bf69ead5ba9--meshery-docs-preview.netlify.app |
Signed-off-by: dusdjhyeon <dusdj0813@gmail.com>
Signed-off-by: dusdjhyeon <dusdj0813@gmail.com>
Signed-off-by: dusdjhyeon <dusdj0813@gmail.com>
f4fa471
to
2f4cf63
Compare
🚀 Preview for commit 2f4cf63 at: https://6674d30fff107bf2733e505a--meshery-docs-preview.netlify.app |
<th>Subcommand</th> | ||
<th>Flag</th> | ||
<th>Subcommand 1</th> | ||
<th>Subcommand 2</th> |
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.
@alphaX86 what do you think?
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.
IMO, I think we can just remove the exp
from table (we can highlight from the heading before table itself) so we can just put it as Command and Subcommand. // @dusdjhyeon
@singh1203 will you review? |
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.
Also, make sure to have other heading consistent
You mean the part about subcommand1 and subcommand2? |
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.
Changes look good but I need further opinion on table format //@singh1203
🚀 Preview for commit f1ef258 at: https://66756d68a79a9674ff798ef8--meshery-docs-preview.netlify.app |
Signed-off-by: dusdjhyeon <dusdj0813@gmail.com>
95b92c5
to
516ab02
Compare
🚀 Preview for commit 516ab02 at: https://6676bff4a58cfaa7520137e1--meshery-docs-preview.netlify.app |
Signed-off-by: dusdjhyeon <dusdj0813@gmail.com>
🚀 Preview for commit 8b70daa at: https://6676c1ca235f9b8dd5a5c3e4--meshery-docs-preview.netlify.app |
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: dusdjhyeon <dusdj0813@gmail.com>
🚀 Preview for commit 930e4fe at: https://6678f2093bf94c1e7e4ac533--meshery-docs-preview.netlify.app |
Signed-off-by: dusdjhyeon <dusdj0813@gmail.com>
🚀 Preview for commit 31e48e7 at: https://6678f555faf0e246b90f779b--meshery-docs-preview.netlify.app |
Signed-off-by: dusdjhyeon <dusdj0813@gmail.com>
🚀 Preview for commit e885092 at: https://6678fce2a58cfa565f0137a1--meshery-docs-preview.netlify.app |
Signed-off-by: dusdjhyeon <dusdj0813@gmail.com>
🚀 Preview for commit 553a69c at: https://667a11ee39fd8c6415fa47f0--meshery-docs-preview.netlify.app |
@dusdjhyeon |
Notes for Reviewers
This PR fixes #11204
This PR fixes #11238
/references/mesheryctl