]> source.dussan.org Git - jquery-ui.git/commitdiff
Readme: Added more details about landing pull requests.
authorScott González <scott.gonzalez@gmail.com>
Wed, 16 May 2012 13:27:26 +0000 (09:27 -0400)
committerScott González <scott.gonzalez@gmail.com>
Wed, 16 May 2012 13:27:26 +0000 (09:27 -0400)
README.md

index f2845df4c86dd35e76225abd06b8b5d7a8452a13..aa47ea1670d57a504e6e78e918e91e0839f06911 100644 (file)
--- a/README.md
+++ b/README.md
@@ -11,6 +11,7 @@ To discuss development with team members and the community, visit the [Developin
 
 For contributors
 ---
+
 If you want to help and provide a patch for a bugfix or new feature, please take
 a few minutes and look at [our Getting Involved guide](http://wiki.jqueryui.com/w/page/35263114/Getting-Involved).
 In particular check out the [Coding standards](http://wiki.jqueryui.com/w/page/12137737/Coding-standards)
@@ -53,9 +54,15 @@ Run grunt.
 
 For committers
 ---
+
 When looking at pull requests, first check for [proper commit messages](http://wiki.jqueryui.com/w/page/12137724/Bug-Fixing-Guide).
 
 Do not merge pull requests directly through GitHub's interface.
+Most pull requests are a single commit; cherry-picking will avoid creating a merge commit.
+It's also common for contributors to make minor fixes in an additional one or two commits.
+These should be squashed before landing in master.
+
+**Make sure the author has a valid name and email address associated with the commit.**
 
 Fetch the remote first: