Make git commit message checking mandatory
This commit is intended to enable the commit message rule checking for
all OpenBMC repositories. It was previously discussed, and several
repos have opted into the current solution without any incident, or
incorrectly-rejected patchsets that I'm aware of.
The very minor issue of CI not rebuilding on a commit message update has
been resolved, so we should be good to enable this project-wide.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I2a845518b18de21d3fd1d71d38c71d3d36038eb2
diff --git a/scripts/format-code.sh b/scripts/format-code.sh
index 5fbe9ae..a8443e0 100755
--- a/scripts/format-code.sh
+++ b/scripts/format-code.sh
@@ -26,14 +26,11 @@
--ignore-words=openbmc-spelling-ignore.txt \
"${DIR}"/.git/COMMIT_EDITMSG
-# Note, this check will be removed once the commit message rules have some usage
-if [[ -f "${DIR}/.openbmc-enforce-gitlint" ]]; then
- # Check for commit message issues
- gitlint \
- --target "${DIR}" \
- --extra-path "${WORKSPACE}/openbmc-build-scripts/config/gitlint/" \
- --config "${WORKSPACE}/openbmc-build-scripts/config/.gitlint"
-fi
+# Check for commit message issues
+gitlint \
+ --target "${DIR}" \
+ --extra-path "${WORKSPACE}/openbmc-build-scripts/config/gitlint/" \
+ --config "${WORKSPACE}/openbmc-build-scripts/config/.gitlint"
cd "${DIR}"