This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author chris.jerdonek
Recipients chris.jerdonek, eric.araujo, ezio.melotti, ncoghlan, pitrou, sandro.tosi, terry.reedy, tshepang
Date 2013-01-03.08:46:52
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1357202812.64.0.986671519246.issue14468@psf.upfronthosting.co.za>
In-reply-to
Content
Content-wise the patch looks pretty good.  I agree with the recommendations.  A couple suggestions though: I would break up the 20 lines of command-line commands.  Right now that chunk is a bit too long to grasp meaningfully.  My suggestion would be to break it up into two sections with text in between: one for applying a patch to 2.x or 3.x, and the other for merging (forward-porting) from 3.x to 3.y, with a textual explanation of what the subsequent chunk of commands will do.

I would also state (or link to) something about forward-porting from 3.x to 3.y and that 2.7 should be kept separate (both of which I think the current patch assumes knowledge of).  I would also say (or link to) something about pushing all branches simultaneously.

Lastly, might it be worth explicitly dividing the Mercurial stuff into separate sections for (1) everyone, and (2) committers?  Putting the committer-specific stuff (e.g. instructions on merging and pushing changes) in a separate section will simplify things for the general contributor.
History
Date User Action Args
2013-01-03 08:46:52chris.jerdoneksetrecipients: + chris.jerdonek, terry.reedy, ncoghlan, pitrou, ezio.melotti, eric.araujo, sandro.tosi, tshepang
2013-01-03 08:46:52chris.jerdoneksetmessageid: <1357202812.64.0.986671519246.issue14468@psf.upfronthosting.co.za>
2013-01-03 08:46:52chris.jerdoneklinkissue14468 messages
2013-01-03 08:46:52chris.jerdonekcreate