Patch Review Process

This page explain the Mercurial Patch Review Process and how (anyone) can help.

1. Generic Fact

• All reviews happen one MailingLists#The_Mercurial-Devel_list

• Contributor follow the ContributingChanges and send they patch to the list (hopefully using the PatchbombExtension)

• Review are just email reply to the emailed patch

• Everyone is welcome to do review.

2. Simple Review Checklist

• The patch is compliant to the ContributingChanges bullet list.

  • Quick reminder of important thing:
  • commit message format,
  • Patch does one and one thing,
  • Change is tested
  • Documentation augmented an updated
  • (all the other things in the list)
• You understand the change
• The change seems correct
• The change seems efficient

If any concerns raised, reply to the email asking question.

If everything sounds good, reply to the email too. Just state it looks good to you.

3. Review Tooling

• Patchwork
• Patch bomb
• Collection of script