Changes between Version 10 and Version 11 of CommitterTips


Ignore:
Timestamp:
Mar 2, 2010, 6:06:25 AM (15 years ago)
Author:
Chris Jerdonek
Comment:

Added section about rolling out.

Legend:

Unmodified
Added
Removed
Modified

  • CommitterTips

    v10 v11  
     1[[PageOutline]]
     2
    13= Tips and Tools for Committers and Reviewers =
    24
    … …  
    7880Monitor [http://build.webkit.org/console] to make sure your patch doesn't turn any of the buildbots red!
    7981
     82== Rolling out a commit ==
     83
     84Here are some guidelines for rolling out (i.e. undoing) a commit.
     85
     86 * Rolling out a commit should undo everything but the ChangeLog entries associated to a commit.
     87 * A commit that rolls out a prior commit should itself contain ChangeLog entries (e.g. prepared using prepare-Changelog).
     88 * The commands `patch -R`, `svn-unapply`, `webkit-patch rollout`, and `git revert` are all helpful for rolling out.
     89
    8090== Guidelines for committers ==
    8191 * Build fixes do not need review.