Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Docs] Add more details to commit message styleguide #1805

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fixup! [Docs] Add more details to commit message styleguide
Signed-off-by: Michał Kowalczyk <mkow@invisiblethingslab.com>
  • Loading branch information
mkow committed Mar 20, 2024
commit 11a3972d24edcb91ba5fd3eba1cc2cfcf9415700
2 changes: 1 addition & 1 deletion CONTRIBUTING.rst
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ Reviewing Guidelines
the same time).
#. Meaningful commit messages (it's much easier to get them right if commits are
really atomic). Commit messages should follow `commit message style guidelines
<coding-style.html#commit-message-formatting>`__.
<https://gramine.readthedocs.io/en/latest/devel/coding-style.html#commit-message-formatting>`__.
#. Every PR description should include: what's the purpose of the changes, what
is changed (and how, in case of redesigning a component), and how to test the
changes.
Expand Down
7 changes: 4 additions & 3 deletions Documentation/devel/coding-style.rst
Original file line number Diff line number Diff line change
Expand Up @@ -43,16 +43,17 @@ Commit message formatting
TL;DR: we follow https://commit.style/.

Additionally, commit message one-liner should include which component was
changed (``PAL-{Linux,SGX}`` / ``LibOS`` / ``Docs`` / ``CI``) in the format
"[component] Change description". The one-liner should be
changed (``PAL/{Linux,Linux-SGX}`` / ``LibOS`` / ``Docs`` / ``CI``) in the
format "[component] Change description". The one-liner should be
written in imperative mood and say what this particular commit changes.

We aim for our repository and commit history to be self-contained and still
usable even without GitHub (e.g. in case we migrate to a different provider).
Thus, please restrain from descriptions like "Fixes #1234" and instead describe
the fix in the commit message. If you really need to reference something on our
GitHub, then use the full URL instead of #1234, so that it won't break if we
move to a different repository.
move to a different repository. If you want to reference a specific commit,
please quote its one-liner message and optionally its hash.

Please never write descriptions like "file.c doesn't flush buffers on exit" -
it's not clear whether it's what the commit does, or whether it's the problem it
Expand Down