[MERGE][Bug #113436] bzr pull --help should explain the status flags

James Westby jw+debian at jameswestby.net
Fri Jun 8 07:00:05 BST 2007


On (08/06/07 00:38), Daniel Watkins wrote:
> Hi all,
> 
> Attached is a bundle fixing the above bug as well as adding the
> corresponding help to all commands (that I could find) that also use
> status flags, as suggested in the bug report.

Sounds good to me.

> +
> +    Changes that are pulled from the parent branch are summarised using status
> +    flags.  For more information about status flags, see 'help status-flags'.
>      """

> +
> +    Updates from running this will be summarised using status flags.
> +    For more information about status flags, see 'help status-flags'.
>      """

>  
> +    The changes merged are summarised using status flags.  For more information
> +    about status flags, see 'help status-flags'.
> +

These three could be done like this, or using _see_also, but I don't have
a strong opinion. Did you consider this choice?

> +topic_registry.register('status-flags', _status_flags,
> +                        "Help on status flags.")

There, is no test for this, but it's not that important. See
bzrlib/tests/blackbox/test_help.py and the tests for 'help
working-trees' for an easy test to copy.

Otherwise +1 from me.

Thanks,

James

-- 
  James Westby   --    GPG Key ID: B577FE13    --     http://jameswestby.net/
  seccure key - (3+)k7|M*edCX/.A:n*N!>|&7U.L#9E)Tu)T0>AM - secp256r1/nistp256




More information about the bazaar mailing list