Code review comment for lp:~doxxx/bzr/mergetools-commands

Revision history for this message
Vincent Ladeuil (vila) wrote :

>>>>> Gordon Tyler <email address hidden> writes:

    > On 2/9/2011 9:37 AM, Vincent Ladeuil wrote:
    >> Or may be it's really a distinct helper :) Whatever you chose, make
    >> sure a wt object is available, you use only GlobalConfig so far, but
    >> in the long term this would probably be a branch or wt config, both
    >> of which could be accessed via a wt.

    > Is there a way to get a config from a wt object now?

No.

    > I looked but either I'm looking in the wrong place or it's not
    > currently available.

    > Anyway, I've extracted that code out into a helper which takes a
    > mergetool name and config object.

Don't forget to put your proposal status to 'Needs Review' when you're
happy with the result.

      Vincent

« Back to merge proposal