9998-current.patch
9999-current.patch
[LU-3568] Leading comments should be stripped from commit messages Created: 08/Jul/13 Updated: 13/Sep/13 Resolved: 04/Sep/13 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.1.6 |
| Fix Version/s: | Lustre 2.5.0 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | Bruce Korb (Inactive) | Assignee: | Keith Mannthey (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | mn1, patch | ||
| Attachments: |
9998-current.patch
9999-current.patch
|
| Story Points: | 0.1 |
| Rank (Obsolete): | 8979 |
| Description |
|
Sometimes, git likes to insert "git status" commentary at the top of an auto-generated commit message starter file. If the committer carelessly neglects to remove the stuff, the script thinks the message has a missing commit summary line. The fix is trivial. |
| Comments |
| Comment by Bruce Korb (Inactive) [ 08/Jul/13 ] |
|
Cheng suggested an alternative "9998-current.patch" |
| Comment by Bruce Korb (Inactive) [ 08/Jul/13 ] |
|
http://review.whamcloud.com/6928 == tabify commit-msg script |
| Comment by Bruce Korb (Inactive) [ 11/Jul/13 ] |
|
Tabification has now been pushed. The real patch is now built upon this. |
| Comment by Peter Jones [ 13/Jul/13 ] |
|
Keith Could you please take care of this one? Thanks Peter |
| Comment by Keith Mannthey (Inactive) [ 15/Jul/13 ] |
|
These changes have been committed to Master. Is it intended to push these changes to other branches? |
| Comment by Keith Mannthey (Inactive) [ 04/Sep/13 ] |
|
Patch for Master has been landed. |