view CONTRIBUTING @ 1102:93cb29247d61

exchange: check for remote refs to determine if we're cloning The exchange.pull wrapper function was checking whether any paths were configured yet as a proxy to determine whether we're doing the initial clone or a subsequent pull. Core hg @ bdae51a83dfb (released in 4.5) now sets the 'default' path on the ui object before exchange.pull is called, but we can check a bit more directly by checking if we have any remote refs yet. This is the same as a check we do in GitHandler.fetch_pack().
author Kevin Bullock <kbullock@ringworld.org>
date Sat, 03 Feb 2018 14:54:35 -0600
parents 06d523eb596a
children
line wrap: on
line source

The short version:
 * Patches should have a good summary line for first line of commit message
 * Patches should be sent to the Google Group[0]
 * Patch needs to do exactly one thing
 * testsuite passes

The longer version:

We use a variant of Mercurial's own contribution
system. https://www.mercurial-scm.org/wiki/ContributingChanges contains
their full guidelines. Key differences are (by rule number):

 1. For hg-git, I'm not strict about the "topic: a few words" format
    for the first line, but I do insist on a sensible summary as the
    first line of your commit
 2. We don't require use of issueNNN for bug fixes (we have no robot)
 3. Use the hg-git google group
 10. We use mostly pep8 style. The current codebase is a mess, but new
     code should be basically pep8.

0:
Mailing list: hg-git@googlegroups.com
Archives: https://groups.google.com/forum/#!forum/hg-git

Pull requests are generally not noticed more than once every few
months. If you do a pull request, I'm still going to expect you to
have a clean history, and to be willing to rework history so it's
clean before I push the "merge" button.