[ANN] tagmerge - an extension for merging .hgtags

Greg Ward greg-hg at gerg.ca
Sun Apr 4 20:08:08 CDT 2010


On Thu, Apr 1, 2010 at 1:32 AM, Qi Yang <qi.yang137 at gmail.com> wrote:
> I just pushed some change in tagmerge on bitbucket. Please take a look at it
> when you have time.

You will have a much easier time getting your code reviewed if you
just patchbomb the list.  See

http://mercurial.selenic.com/wiki/DeveloperInfo
http://mercurial.selenic.com/wiki/ContributingChanges
http://mercurial.selenic.com/wiki/SuccessfulPatch

> I have some concerns about the change.

Best way to call attention to these is to add comments to your code.
Something like "# XXX is this really the right way?" will get a
reviewer's attention focussed on the right place.

Greg


More information about the Mercurial-devel mailing list