docs: Use install command for kubectl binary on macOS
What this PR does / why we need it
This PR updates the kubectl installation documentation for macOS to use the install command instead of separate chmod, mv, and chown commands.
Before (3 commands, 2 steps):
chmod +x ./kubectl
sudo mv ./kubectl /usr/local/bin/kubectl
sudo chown root: /usr/local/bin/kubectl
After (1 command, 1 step):
sudo install -o root -m 0755 ./kubectl /usr/local/bin/kubectl
The install command is cleaner because it:
- Sets executable permissions (
-m 0755) - Sets ownership (
-o root) - Copies the file to destination
All in a single, explicit command.
Which issue(s) this PR fixes
Fixes https://github.com/kubernetes/website/issues/53518 (partial - macOS page only)
Special notes for your reviewer
- This PR only updates the macOS installation page as a starting point
- The same pattern exists in ~35 other files (as noted in the issue)
- If this approach is approved, similar updates can be made to other installation docs
/kind documentation
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign divya-mohan0209 for approval. For more information see the Code Review Process.
The full list of commands accepted by this bot can be found here.
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
Pull request preview available for checking
Built without sensitive environment variables
| Name | Link |
|---|---|
| Latest commit | e8807b6ae467bd9c44fe6ca715b3386d30675d68 |
| Latest deploy log | https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/696abab9a87799000805408d |
| Deploy Preview | https://deploy-preview-54045--kubernetes-io-main-staging.netlify.app |
| Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify project configuration.
Not tested (no suitable Mac) but the changes look right to me.
Hi can we merge this PR if this is looking good. Thanks!
Hi @mikedanese @natalisucks requesting to review this PR and let me know if there is anything needed. Thanks