-
Notifications
You must be signed in to change notification settings - Fork 206
Commit Message Format
John Haddon edited this page May 20, 2021
·
4 revisions
The Gaffer project has dragged on for some time, but has not yet descended into chaos. This page documents our standards for commit messages.
- Start with a one-line subject, no longer than 70 characters. This fits inside GitHub's commit view without being truncated.
- Start the subject with the name of the class or module being edited, followed by a colon. For example, "ImageTransform :" for edits to the ImageTransform node or "GafferScene :" for broad changes to the scene module.
- Use the imperative mood rather than the past tense. For example "Fix bug..." not "Fixed bug...". This may feel awkward at first, but matches standard Git practice and anecdotally makes it easier to compose meaningful messages inside the 70 character limit.
- Leave an empty line between the subject and the body.
- Explain what is being changed and why, so that a fellow developer has enough information to evaluate the change.
- Include
Fixes #100
to close any issues fixed by the commit.