Re: svn commit: r1297921 - /subversion/trunk/subversion/tests/cmdline/svntest/sandbox.py

Philip Martin <philip.martin_at_wandisco.com> writes:

> Greg Stein <gstein_at_gmail.com> writes:
>
>> Geez. Be smart, Philip. Obviously, that quoted content is dumb. I'm talking
>> about format. Specifically not following the basic rule of "one line
>> summary. One blank line. Details." Julian's change is missing the blank
>> lines, which many tools parse.
>
> That's what I don't understand. Are we expecting people to use tools to
> parse these docstrings? If we have to repeat default parameters in the
> docstring it it something we can get wrong, if we don't repeat them they
> can't be wrong. What do we gain?

Maybe what you want is that we follow some of PEP-257 but not all of it?
How should the docstring change?

    """Append CONTENTS to file DEST, optionally truncating it first.
       DEST is a relpath relative to the WC."""

-- 
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com