Skip to content
This repository has been archived by the owner on Nov 30, 2022. It is now read-only.

Add Forking Clarification #22

Merged
merged 2 commits into from
Nov 2, 2021
Merged

Add Forking Clarification #22

merged 2 commits into from
Nov 2, 2021

Conversation

pattisdr
Copy link
Contributor

@pattisdr pattisdr commented Nov 2, 2021

Purpose

Dev docs specify to clone the original, but most contributors will be cloning a fork.

Changes

  • Clarify at the top of the docs that you should be cloning your fork
  • Add some more specific instructions about how to keep your fork in sync with the upstream branch
  • Also, make the PR from my fork!

Ticket

[unticketed]

@NevilleS
Copy link
Contributor

NevilleS commented Nov 2, 2021

Good point!

@NevilleS NevilleS merged commit 9ec8349 into ethyca:main Nov 2, 2021
sanders41 pushed a commit that referenced this pull request Sep 22, 2022
* Add more instructions for forking the repo beyond just linking to Github docs.

* Fix formatting.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants