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

To fork or not to fork? #31

Closed
thinkingfish opened this issue Nov 8, 2015 · 5 comments
Closed

To fork or not to fork? #31

thinkingfish opened this issue Nov 8, 2015 · 5 comments
Labels

Comments

@thinkingfish
Copy link
Contributor

Shall we each clone the repo under our own account, and submit pull requests from there, or shall we continue to create branches under the main repo?

Only those with write-access to the main repo can create branches, which means the workflow is different for those who have this access and those who don't. On the other hand, it is easier to see what branches are outstanding when they are all in one place, before a PR is created, and it doesn't require we modify how we setup our local repo yet again.

@thinkingfish

This comment was marked as spam.

@seppo0010
Copy link
Contributor

We should fork, but once it is open source.

@sagar0
Copy link
Contributor

sagar0 commented Nov 8, 2015

Based on Forking Projects guide, I get the feeling that if you have write access to master (i.e. a maintainer), then forking is not necessary. But for any one who is not on the 'maintainers' list and want to contribute to the project, the only way to do that is to fork and create a pull request.

I don't have much experience on Github collaboration to vote on the issue, but may be we should talk to other twitter folks too who maintain major twitter repositories (say, finagle or scalding).

@sagar0
Copy link
Contributor

sagar0 commented Nov 21, 2015

+1 to fork.
After getting some more experience with the github work flow, I am all for forking. In my personal workflow, I tend to create a ton of branches for every minor idea that I get. Plus I am paranoid about my hard disk crashing, due to which I tend to push all my local branches to origin. I don't want to pollute twitter/pelikan repo with all my crappy branches.

@seppo0010
Copy link
Contributor

👍 let's do that. I'll use fork for all my new changes.

michalbiesek added a commit to michalbiesek/pelikan that referenced this issue Jul 3, 2019
swlynch99 pushed a commit to swlynch99/pelikan-twitter that referenced this issue Sep 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants