Skip to content
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

change clone instructions from git to https protocol #5526

Merged
merged 1 commit into from
Oct 12, 2023

Conversation

MikeMcC399
Copy link
Contributor

@MikeMcC399 MikeMcC399 commented Oct 8, 2023

This PR resolves an issue that attempting to clone the repository may result in the error "Permission denied (publickey).".

Issue

CONTRIBUTING.md > Writing Documentation > Clone your forked repository contains the following instructions to "Clone your forked repository":

git clone [email protected]:<your username>/cypress-documentation.git
cd cypress-documentation

This may result in a failure to clone the forked repo. For example:

$ git clone [email protected]:MikeMcC399/cypress-documentation.git
Cloning into 'cypress-documentation'...
[email protected]: Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

The cloning instructions use SSH syntax and require an SSH certificate to work. This is not described in the instructions.

Change

The instructions are changed to use the https protocol, which is widely available and requires no other preparation:

Previously

git clone [email protected]:<your username>/cypress-documentation.git

changed to

git clone https://github.com/<your username>/cypress-documentation.git

Reference

@netlify
Copy link

netlify bot commented Oct 8, 2023

👷 Deploy request for cypress-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 605ff19

@cypress-app-bot
Copy link
Collaborator

@netlify
Copy link

netlify bot commented Oct 8, 2023

Deploy Preview for benevolent-cat-040f48 ready!

Name Link
🔨 Latest commit 605ff19
🔍 Latest deploy log https://app.netlify.com/sites/benevolent-cat-040f48/deploys/65263fd3c49d3e000890c0fd
😎 Deploy Preview https://deploy-preview-5526--benevolent-cat-040f48.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 site configuration.

@MikeMcC399 MikeMcC399 marked this pull request as ready for review October 8, 2023 13:01
@mschile mschile merged commit 9a8f5a7 into cypress-io:main Oct 12, 2023
@mschile mschile self-assigned this Oct 12, 2023
@MikeMcC399 MikeMcC399 deleted the clone-instructions branch October 12, 2023 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants