Improve kubectl introduction for clarity#55304
Improve kubectl introduction for clarity#55304sobit02 wants to merge 1 commit intokubernetes:mainfrom
Conversation
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
Caesarsage
left a comment
There was a problem hiding this comment.
Thanks for your PR. I think this is good for beginners. Left a few comments
|
Thanks for the review and helpful suggestions! I agree that using the glossary definition for the synopsis is better to avoid duplication. I’ll also update the introduction to use more active voice as suggested. I’ll make these changes and push an update in a few hours. |
|
Thanks for the suggestions! I've updated the introduction to use active voice and replaced the synopsis with the |
|
Thanks, this look good. can you help squash your commits. |
6a43c73 to
a5584e5
Compare
a5584e5 to
7ec8654
Compare
|
Squashed the commits into a single commit and updated the PR. Please review and let me know if anything else is needed. |
|
Refined the JSONPath description to improve grammatical correctness and align with the existing documentation style. |
|
Seems the PR now contains some unrelated changes. Kindly take a look at it. Thanks |
|
Thanks for pointing that out. I've removed the unrelated JSONPath changes and kept this PR focused on improving the kubectl introduction. |
41d41a7 to
20f5f73
Compare
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Thanks for the feedback. I’ve rebased the branch on upstream/main and ensured the PR now only contains the intended changes to kubectl documentation. |
|
/lgtm Thank you |
|
LGTM label has been added. DetailsGit tree hash: 112b4aa804aea7f07df3b1a6c67fd789f72f7fc1 |
|
/remove-language zh |
|
/remove-area localization |
What I did
Why