|
Size: 3227
Comment: reference to --partial
|
Size: 3744
Comment:
|
| Deletions are marked like this. | Additions are marked like this. |
| Line 1: | Line 1: |
| #pragma section-numbers 2 <<Include(A:style)>> <<Include(A:dev)>> |
|
| Line 5: | Line 10: |
| <<TableOfContents>> | |
| Line 28: | Line 34: |
| * Documentation augmented an updated | * Documentation augmented and updated |
| Line 45: | Line 51: |
| Some people are blessed to accept patches and push them to a repo where Matt Mackall ultimatly pull from. | Some people are blessed to accept patches and push them to a repo where Matt Mackall ultimately pulls from. |
| Line 55: | Line 61: |
---- /!\ '''Edit conflict - other version:''' ---- * you can get the patches files directly from http://review.octopoid.net/patches/<node>/raw.txt Appropriate hg alias would be: |
* you can get the patches files directly from http://hgpatches.durin42.com/patches/<node> Appropriate hg alias would be: |
| Line 61: | Line 65: |
| getpatch=import --partial --obsolete http://review.octopoid.net/patches/$1/raw.txt | getpatch=import --partial --obsolete http://hgpatches.durin42.com/patches/$1 |
| Line 64: | Line 68: |
---- /!\ '''Edit conflict - your version:''' ---- ---- /!\ '''End of edit conflict''' ---- |
|
| Line 71: | Line 71: |
| hg import --partial --obsolete <patches>: }}} |
|
| Line 72: | Line 74: |
| ---- /!\ '''Edit conflict - other version:''' ---- hg import --partial --obsolete <patches>: ---- /!\ '''Edit conflict - your version:''' ---- hg import --partial --obsolete <patches>: ---- /!\ '''End of edit conflict''' ---- }}} |
|
| Line 90: | Line 84: |
| == Patchwork States == ||'''New''' || || ||'''Pre-Reviewed''' ||non-reviewer have "lgtm", but still needs someone to look at it || ||'''Under Review''' || || ||'''2nd Review Requested'''||reviewer looked at it, but second pair of eyes requested || ||'''Accepted''' || || ||'''Changes Requested''' ||changes requested by reviewer, needs new version || ||'''Rejected''' || || ||'''RFC''' ||an RFC patch, needs more reviews (?) || ||'''Superseded''' ||new version available || ||'''Not Applicable''' ||not a patch (?) || ||'''Deferred''' ||? || |
|
| Line 98: | Line 107: |
| * Various data collection [[http://review.octopoid.net/]] (STALED) | |
| Line 99: | Line 109: |
| ---- /!\ '''Edit conflict - other version:''' ---- * Various data collection [[http://review.octopoid.net/]] |
* Matt Mackall Inbox [[http://www.selenic.com/inbox|Metrix (nb email, nb patches, oldest email (in day))]] and [[http://www.selenic.com/inflight|Content]]. |
| Line 102: | Line 111: |
| * [[http://www.selenic.com/inbox|Matt Mackall Inbox Metrix]] (nb email, nb patches, oldest email (in day)) | == The Committers Group == |
| Line 104: | Line 113: |
| ---- /!\ '''Edit conflict - your version:''' ---- * [[http://www.selenic.com/inbox|Matt Mackall Inbox Metrix]] (nb email, nb patches, oldest email (in day)) |
Current list with push access to the [[https://www.mercurial-scm.org/repo/hg-committed/|hg-committed repository]] |
| Line 107: | Line 115: |
| ---- /!\ '''End of edit conflict''' ---- | * Kevin Bullock * Pierre-Yves David * Augie Fackler * Matt Mackall * Yuya Nishihara * Bryan O'Sullivan * Martin von Zweigbergk ---- CategoryDeveloper |
|
This page does not meet our Note:
This page is primarily intended for developers of Mercurial.
Patch Review Process
This page explains the Mercurial patch review process and how (anyone) can help.
Contents
1. Generic Fact
All reviews happen on MailingLists#The_Mercurial-Devel_list
Contributors follow the ContributingChanges and send their patch(es) to the list (hopefully using the PatchbombExtension)
- Reviews are just email replies to the emailed patch
Everyone is welcome to do review.
2. Simple Review Checklist
The patch should conform to the ContributingChanges bullet list.
- Quick reminder of important things:
- commit message format,
- Patch does one and only one thing,
- Change is tested
- Documentation augmented and 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 questions.
If everything sounds good, reply to the email too. Just state it looks good to you.
3. Accepters Review Checklist
Some people are blessed to accept patches and push them to a repo where Matt Mackall ultimately pulls from.
Here is another check list for them
- Run check code on all patches
- Run the whole test suites
- Reply to the list saying that you took care of the patch
you can get the patches files directly from http://hgpatches.durin42.com/patches/<node> Appropriate hg alias would be:
[alias] getpatch=import --partial --obsolete http://hgpatches.durin42.com/patches/$1
- Make sure you created obsolescence marker between the node in the patch and the one you created, e.g.
hg import --partial --obsolete <patches>:
use the drophack extension if you need to drop a changeset you queued
Rebase your queue on top of main's @
Move @ with the changeset you pushed.
Update Patchwork once you have pushed
4. Patchwork States
New |
|
Pre-Reviewed |
non-reviewer have "lgtm", but still needs someone to look at it |
Under Review |
|
2nd Review Requested |
reviewer looked at it, but second pair of eyes requested |
Accepted |
|
Changes Requested |
changes requested by reviewer, needs new version |
Rejected |
|
RFC |
an RFC patch, needs more reviews (?) |
Superseded |
new version available |
Not Applicable |
not a patch (?) |
Deferred |
? |
5. Review Tooling
Various data collection http://review.octopoid.net/ (STALED)
Matt Mackall Inbox Metrix (nb email, nb patches, oldest email (in day)) and Content.
6. The Committers Group
Current list with push access to the hg-committed repository
- Kevin Bullock
- Pierre-Yves David
- Augie Fackler
- Matt Mackall
- Yuya Nishihara
- Bryan O'Sullivan
- Martin von Zweigbergk