-
Notifications
You must be signed in to change notification settings - Fork 174
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
Comments
This comment was marked as spam.
This comment was marked as spam.
We should fork, but once it is open source. |
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). |
+1 to fork. |
👍 let's do that. I'll use fork for all my new changes. |
Fix set user data and style issue
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.
The text was updated successfully, but these errors were encountered: