aboutsummaryrefslogtreecommitdiff
path: root/docs/CONTRIBUTE
diff options
context:
space:
mode:
authorDaniel Stenberg <daniel@haxx.se>2012-03-05 23:42:07 +0100
committerDaniel Stenberg <daniel@haxx.se>2012-03-05 23:42:07 +0100
commit4d71d1b17f8179b386c933827cad408da368999a (patch)
treee5a082b47be22fa552b5223062b71d814ff951e1 /docs/CONTRIBUTE
parentd185960a6d56a5298ba9a7ca97683d256f92317c (diff)
CONTRIB: Please don't send pull requests
Diffstat (limited to 'docs/CONTRIBUTE')
-rw-r--r--docs/CONTRIBUTE25
1 files changed, 25 insertions, 0 deletions
diff --git a/docs/CONTRIBUTE b/docs/CONTRIBUTE
index 53e484284..262af4886 100644
--- a/docs/CONTRIBUTE
+++ b/docs/CONTRIBUTE
@@ -34,6 +34,7 @@
3.3 How To Make a Patch without git
3.4 How to get your changes into the main sources
3.5 Write good commit messages
+ 3.6 Please don't send pull requests
==============================================================================
@@ -276,3 +277,27 @@
and make sure that you have your own user and email setup correctly in git
before you commit
+3.6 Please don't send pull requests
+
+ With git (and expecially github) it is easy and tempting to send a pull
+ request to one or more people in the curl project to have changes merged this
+ way instead of mailing patches to the curl-library mailing list.
+
+ We don't like that. We want them mailed for these reasons:
+
+ - Peer review. Anyone and everyone on the list can review, comment and
+ improve on the patch. Pull requests limit this ability.
+
+ - Anyone can merge the patch into their own trees for testing and those who
+ have push rights can push it to the main repo. It doesn't have to be anyone
+ the patch author knows beforehand.
+
+ - Commit messages can be tweaked and changed if merged locally instead of
+ using github. Merges directly on github requires the changes to be perfect
+ already, which they seldomly are.
+
+ - Merges on github prevents rebases and even enforces --no-ff which is a git
+ style we don't otherwise use in the project
+
+ However: once patches have been reviewed and deemed fine on list they are
+ perfectly OK to be pulled from a published git tree.