Probably the point in adding submitted by, is for submitted patches. A
lot of contributors don't have commit rights so their names wouldn't
show up from the SCM if you view the logs.
Carlos Sanchez wrote:
Sorry, when I said issue number I meant source control commit id, so
when you ask for the comment used you can also get the user from the
SCM
On 10/27/05, Brett Porter <[EMAIL PROTECTED]> wrote:
I think it is a good idea to include it in the SVN commit message (also
a description of what was going on). It makes the source control stand
on its own as a source of information. Otherwise, if you later analyse
the history and it points to JIRA when we've moved on to
ReallyGoodIssueTracker2010, its a bit hard to know who submitted it.
- Brett
Carlos Sanchez wrote:
I don't see the point of including the submitted by, probably it can
be obtained in any other way from the issue number. I agree with the
jira issue number
On 10/27/05, Brett Porter <[EMAIL PROTECTED]> wrote:
Hi,
I've noticed a few things going into commit messages, for which I
usually use a standard form (that matches a template that used to exist
in Apache's CVS).
Basically, wehere applicable:
PR: issue1, issue2 (using issue keys from JIRA, not URLs)
Submitted by: (when it was a patch, put that persons name there)
Reviewed by: (not sure if this is useful, but was in the template - who
reviewed it)
<< comment >>
What do folks think of using this? I just find it a bit easier to parse
when scanning, and also as we move to get JIRA including all our commits
linked in issues the first part is especially helpful.
Just a thought... don't know if we can have a template for SVN commit
messages like CVS though.
- Brett
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
