Skip to content

Add 'https://' to Fix Failing 'git clone' Command - #59123

Open
stevesynk wants to merge 1 commit into
kubernetes:mainfrom
stevesynk:fix-ref-docs-clone
Open

stevesynk wants to merge 1 commit into
kubernetes:mainfrom
stevesynk:fix-ref-docs-clone

Conversation

@stevesynk

@stevesynk stevesynk commented Nov 8, 2026 •

Copy link
Copy Markdown
Contributor

Description

While working on API docs, I noticed that the guide for generating API ref docs has a git clone command that fails when you try to run it in the terminal. The URL needs 'https://' to work. I tested it, and with that change the git clone command works properly.

Issue

no previous issue that I'm aware of.

AI disclosure -- I use Claude to help me with git commands while editing in VS Code. All writing is my own and no information has been included that I did not personally verify.

@kubernetes-prow

Copy link
Copy Markdown
Contributor

[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 nate-double-u for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubernetes-prow kubernetes-prow Bot added the language/en Issues or PRs related to English language label Nov 8, 2026
@kubernetes-prow kubernetes-prow Bot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 8, 2026
@netlify

netlify Bot commented Nov 8, 2026

Copy link
Copy Markdown

✅ Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 217ae14
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/6af0ea1dfe4706a7188e6ef0
😎 Deploy Preview https://deploy-preview-59123--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@stevesynk

Copy link
Copy Markdown
Contributor Author

@katcosgrove @nate-double-u

This one is ready for review, and should be super straightforward. I just fixed a git command from the API ref docs guide that wasn't working.

thanks!

@dipesh-rawat dipesh-rawat left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@kubernetes-prow kubernetes-prow Bot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 9, 2026
@kubernetes-prow

Copy link
Copy Markdown
Contributor

LGTM label has been added.

DetailsGit tree hash: 5204f10280d38fba55b13565fb31b757230d1534

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants