* simplifies some sentences
* use a more neutral language with less you/yours
This commit is contained in:
Xavier Noria
2011-03-05 11:29:57 +01:00
parent 5f99a899a8
commit f6a5449d43

View File

@@ -349,22 +349,21 @@ $ git commit -a
$ git format-patch master --stdout > my_new_patch.diff
</shell>
Sanity check the results of this operation: open the diff file in your text editor of choice and make sure that no unintended changes crept in.
Open the diff file in your text editor of choice to sanity check the results, and make sure that no unintended changes crept in.
You can check your patches by applying your patch to an different dedicated branch:
You can also perform an extra check by applying the patch to a different dedicated branch:
<shell>
$ git checkout -b testing_branch
$ git apply --check my_new_patch.diff
</shell>
You can make sure your patches don't add any whitespace by applying it yourself using the --whitespace=error-all option. Make sure you are on your dedicated test branche and:
Please make sure the patch does not introduce whitespace errors:
<shell>
$ git apply --whitespace=error-all mynew_patch.diff
</shell>
h4. Create a Lighthouse Ticket
Now create a ticket with your patch. Go to the "new ticket":http://rails.lighthouseapp.com/projects/8994-ruby-on-rails/tickets/new page at Lighthouse. Fill in a reasonable title and description, remember to attach your patch file, and tag the ticket with the patch tag and whatever other subject area tags make sense.