[MERGE] show merge indicators in log --long

Vincent Ladeuil v.ladeuil+lp at free.fr
Fri Mar 27 13:20:56 GMT 2009


BB:comment

Lacking time to review correctly.

>>>>> "Ian" == Ian Clatworthy <ian.clatworthy at internode.on.net> writes:

<snip/>

    Ian> +    def show_advice(self):
    Ian> +        """Output user advice, if any, when the log is completed."""
    Ian> +        if self.levels == 1 and self._merge_count > 0:
    Ian> +            advice_sep = self.get_advice_separator()
    Ian> +            if advice_sep:
    Ian> +                self.to_file.write(advice_sep)
    Ian> +            self.to_file.write("Use -n0 to see merged revisions.\n")

Can we use ' Use --levels 0 to see merged revisions.\n' ?

I think it carries more information and could a void a 'bzr help
log' trip.

Ans as a rule I think we should never use short names or aliases
in documentation.

<snip/>

    Ian> === modified file 'bzrlib/tests/blackbox/test_log.py'
    Ian> --- bzrlib/tests/blackbox/test_log.py	2009-03-26 05:41:59 +0000
    Ian> +++ bzrlib/tests/blackbox/test_log.py	2009-03-27 01:53:11 +0000

Don't you love localized tests ? :-P

      Vincent



More information about the bazaar mailing list