Skip to content

Conversation

@ag-tafe
Copy link
Contributor

@ag-tafe ag-tafe commented Oct 26, 2024

Fixes #18050

Makes cloning of the repository easier for new starters. It allows to skip a complicated step of generating ssh keys, but provides a link to detailed instructions on how to do so.

@ag-tafe ag-tafe changed the title Update repository clone method. Fixes issue 18050. [Documentation] Update repository clone method. Fixes issue 18050 Oct 26, 2024

```bash
git clone git@github.com:<your_username>/mypy.git
git clone https://github.com/<your_username>/mypy.git
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think this is a good idea given that this is the contributing guide where people would want to push to the repository but cannot with a https clone. Maybe add the link to GitHub shh docs but keep this command unchanged?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@hamdanal No problem at all. I'm just still a newbie about github. If I'm to amend this pull request should I do it locally and then push onto my fork of mypy for the pull request to automatically amend or just cancel this pull request and create a new one?

Copy link
Member

@brianschubert brianschubert Oct 27, 2024

Choose a reason for hiding this comment

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

@ag-tafe The former. You can update this PR by making changes locally and pushing to the branch in your fork. Any new commits will appear in the PR history.

In case helpful as a reference, this is what the cloning instructions look like in CPython's contribution guide: https://devguide.python.org/getting-started/setup-building/

(btw: it looks like your commit email is associated with a different github account than the one you opened this PR with)

@hauntsaninja
Copy link
Collaborator

Closing as stale

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.

Repository cloning via ssh throws an error

5 participants