You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@trafficserver.apache.org by Leif Hedstrom <zw...@apache.org> on 2010/05/17 22:29:51 UTC

[discuss] Commit policies

Hi all,

I'd like to open up the discussion on maybe having some "ground rules" 
for how we commit, build and test on trunk. I've started a draft on

     https://cwiki.apache.org/confluence/display/TS/CommitPolicies


and a few people have already made some modifications to it as well.

Please discuss :).

-- leif


Re: [discuss] Commit policies

Posted by Leif Hedstrom <zw...@apache.org>.
I think we've finished this discussion, and there has been a lot of 
editing to the Wiki page. Unless someone objects (-1), the Wiki page now 
captures our commit policies:

     https://cwiki.apache.org/confluence/display/TS/CommitPolicies


Cheers,

-- leif


Re: [discuss] Commit policies

Posted by Rainer Jung <ra...@kippdata.de>.
On 25.06.2010 19:19, Andrew Hsu wrote:
> FYI: I've added a section for describing the 'Author:' field of a commit
> log, mostly to remind myself of proper attribution upon commit :)
> https://cwiki.apache.org/confluence/display/TS/CommitPolicies

IMHO it is generally a good practice to obfuscate the email address, 
e.g. by replacing "@" and dots in the domain name by spaces. See e.g. 
the httpd changes file.

Of course that's not perfect.

Regards,

Rainer

Re: [discuss] Commit policies

Posted by Andrew Hsu <an...@apache.org>.
FYI: I've added a section for describing the 'Author:' field of a commit
log, mostly to remind myself of proper attribution upon commit :)
https://cwiki.apache.org/confluence/display/TS/CommitPolicies

Re: [discuss] Commit policies

Posted by John Plevyak <jp...@acm.org>.
+1

On 5/17/2010 6:20 PM, Leif Hedstrom wrote:
> On 05/17/2010 06:11 PM, Andrew Hsu wrote:
>> It may be useful to have a policy about commit log formats as well. 
>> An example of a commit log with jira ticket number that displays well
>> with git:
>>
>> TS-#### short summary (50 chars or less), not a sentence
>>
>> More detailed explanatory text, if necessary.  Wrap it to about 72
>> characters or so.  In some contexts, the first line is treated as the
>> subject of an email and the rest of the text as the body.  The blank
>> line separating the summary from the body is critical (unless you omit
>> the body entirely); tools like rebase can get confused if you run the
>> two together.
> 
> +1, add it to the Cwiki.
> 
> -- leif


Re: [discuss] Commit policies

Posted by Leif Hedstrom <zw...@apache.org>.
On 05/17/2010 06:11 PM, Andrew Hsu wrote:
> It may be useful to have a policy about commit log formats as well.  
> An example of a commit log with jira ticket number that displays well 
> with git:
>
> TS-#### short summary (50 chars or less), not a sentence
>
> More detailed explanatory text, if necessary.  Wrap it to about 72
> characters or so.  In some contexts, the first line is treated as the
> subject of an email and the rest of the text as the body.  The blank
> line separating the summary from the body is critical (unless you omit
> the body entirely); tools like rebase can get confused if you run the
> two together.

+1, add it to the Cwiki.

-- leif


Re: [discuss] Commit policies

Posted by Andrew Hsu <an...@apache.org>.
It may be useful to have a policy about commit log formats as well.  An 
example of a commit log with jira ticket number that displays well with git:

TS-#### short summary (50 chars or less), not a sentence

More detailed explanatory text, if necessary.  Wrap it to about 72
characters or so.  In some contexts, the first line is treated as the
subject of an email and the rest of the text as the body.  The blank
line separating the summary from the body is critical (unless you omit
the body entirely); tools like rebase can get confused if you run the
two together.

On 5/17/10 1:29 PM, Leif Hedstrom wrote:
> Hi all,
>
> I'd like to open up the discussion on maybe having some "ground rules" 
> for how we commit, build and test on trunk. I've started a draft on
>
>     https://cwiki.apache.org/confluence/display/TS/CommitPolicies
>
>
> and a few people have already made some modifications to it as well.
>
> Please discuss :).
>
> -- leif
>