-
1. Git commit messages
tfennelly May 11, 2011 5:27 AM (in response to kcbabo)3 small comments:
- How about adding the full JIRA url to the issue? I like to add full URLs for things anywhere possible... git commits, pull requests etc. Makes it convenient.
- Short description should match that of the JIRA.
- Is there really a need to add the long description? The short description should give enough info unless it's really bad, no? On the rare occasion you might need more info, then follow the link to the JIRA (see#1 above ).
-
2. Re: Git commit messages
kcbabo May 11, 2011 6:44 AM (in response to tfennelly)1. How about adding the full JIRA url to the issue? I like to add full URLs for things anywhere possible... git commits, pull requests etc. Makes it convenient.
Yeah, this is really the only difference between the way you do it and the way I do it. Personally, I find that the full URL tends to take up valuable room in my one-line commit message. Also, I know the root JIRA URL where I can just append the JIRA id. Now, I agree that not all people will have this knowledge, so I'm willing to go either way (full URL or just the id). To get an idea of what I'm talking about, here's a selection of recent commits:
03dc299dddaa05e97834f9e526f5ae75e58576b8 https://issues.jboss.org/browse/SWITCHYARD-178 TorqueBox POC - using SwitchYard f38ef2b7693fef53a87d66c755f5743a6a21f5b6 SWITCHYARD-176 Initial implementation of Forge support for bean, soap, and came 1d9b10d967b5972e1796362faffb9a26a1ba2386 SWITCHYARD-189 Update components based on deployment and activator changesbb34b0b1368f80adf6cae877cda133b34da41d43 SWITCHYARD-83 Updated component code to adjust to core API changes c9ed7a3dd16a989edb6af843ff42c1e394c80a07 SWITCHYARD-205 Fix checkstyle issues 0b865a862c3f21f54ddf84e9806d2a4b127ad92e https://issues.jboss.org/browse/SWITCHYARD-203 Merge @SwitchYardDeploymentScann 47ba22ad16a5f060b9a540442eec02f1ff00cea5 https://issues.jboss.org/browse/SWITCHYARD-185 Wire transformer implementation
2. Short description should match that of the JIRA.
Absoltely agree. Only exception is if the JIRA description is really long.
3. Is there really a need to add the long description? The short description should give enough info unless it's really bad, no? On the rare occasion you might need more info, then follow the link to the JIRA (see#1 above ).
No, the long description is completely optional. Only reason I mention it is to reduce the size of the one-line commit message which is what gets printed in both the GItHub commit history and the git log output with one-line pretty printing. Bottom line is that if you have a lot to say in the commit message (e.g. special build instructions), then do that in the long description.
-
3. Git commit messages
mageshbk May 12, 2011 12:42 AM (in response to kcbabo)I vote for just the id. The rest is agreable.
-
4. Git commit messages
aamonten May 12, 2011 3:24 PM (in response to kcbabo)my preference is just the ID, but its also ok with the URL.
-
5. Git commit messages
antollinim May 18, 2011 4:02 PM (in response to kcbabo)I aggre with the schema proposed by Keith.
Mandatory: [JIRA #] [Short Description]
Optional: [Long Description]