diff mbox

[dpdk-dev] scripts: skip capitalization check for commit prefixes

Message ID 1484312537-22011-1-git-send-email-bruce.richardson@intel.com (mailing list archive)
State Accepted, archived
Headers

Checks

Context Check Description
ci/Intel compilation success Compilation OK

Commit Message

Bruce Richardson Jan. 13, 2017, 1:02 p.m. UTC 
  The prefix in the commit title must be a valid component name and is
checked in separate checks. For capitalization, just check the part after
the colon. This is already done for most capitalization checks, just make
the remainder consistent with this.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 scripts/check-git-log.sh | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Thomas Monjalon Jan. 13, 2017, 4:08 p.m. UTC | #1 
2017-01-13 13:02, Bruce Richardson:
> The prefix in the commit title must be a valid component name and is
> checked in separate checks. For capitalization, just check the part after
> the colon. This is already done for most capitalization checks, just make
> the remainder consistent with this.
> 
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>

Applied, thanks
diff mbox

Patch

diff --git a/scripts/check-git-log.sh b/scripts/check-git-log.sh
index f79f0a2..f6a35d2 100755
--- a/scripts/check-git-log.sh
+++ b/scripts/check-git-log.sh
@@ -113,10 +113,10 @@  bad=$(echo "$headlines" | grep --color=always \
 
 # check headline uppercase (Rx/Tx, VF, L2, MAC, Linux, ARM...)
 bad=$(echo "$headlines" | grep -E --color=always \
-	-e '\<(rx|tx|RX|TX)\>' \
-	-e '\<[pv]f\>' \
-	-e '\<[hsf]w\>' \
-	-e '\<l[234]\>' \
+	-e ':.*\<(rx|tx|RX|TX)\>' \
+	-e ':.*\<[pv]f\>' \
+	-e ':.*\<[hsf]w\>' \
+	-e ':.*\<l[234]\>' \
 	-e ':.*\<api\>' \
 	-e ':.*\<arm\>' \
 	-e ':.*\<armv7\>' \