scripts: improve formating

add git lint, script to install git hook, modified formatting cfg

Signed-off-by: Robert Gałat <robert.galat@nordicsemi.no>

Author: RobertGalatNordic

.gitlint

@@ -0,0 +1,60 @@
+# All these sections are optional, edit this file as you like.
+[general]
+ignore=title-trailing-punctuation, T3, title-max-length, T1, body-hard-tab, B3, B1
+# verbosity should be a value between 1 and 3, the commandline -v flags take precedence over this
+verbosity = 3
+# By default gitlint will ignore merge commits. Set to 'false' to disable.
+ignore-merge-commits=false
+ignore-revert-commits=false
+ignore-fixup-commits=false
+ignore-squash-commits=false
+# Enable debug mode (prints more output). Disabled by default
+debug = false
+
+# Set the extra-path where gitlint will search for user defined rules
+# See http://jorisroovers.github.io/gitlint/user_defined_rules for details
+extra-path=../zephyr/scripts/gitlint
+
+[title-max-length-no-revert]
+line-length=75
+
+[body-min-line-count]
+min-line-count=1
+
+[body-max-line-count]
+max-line-count=200
+
+[title-starts-with-subsystem]
+regex = ^(?!subsys:)(([^:]+):)(\s([^:]+):)*\s(.+)$
+
+[title-must-not-contain-word]
+# Comma-separated list of words that should not occur in the title. Matching is case
+# insensitive. It's fine if the keyword occurs as part of a larger word (so "WIPING"
+# will not cause a violation, but "WIP: my title" will.
+words=wip
+
+[title-match-regex]
+# python like regex (https://docs.python.org/2/library/re.html) that the
+# commit-msg title must be matched to.
+# Note that the regex can contradict with other rules if not used correctly
+# (e.g. title-must-not-contain-word).
+#regex=^US[0-9]*
+
+[max-line-length-with-exceptions]
+# B1 = body-max-line-length
+line-length=72
+
+[body-min-length]
+min-length=3
+
+[body-is-missing]
+# Whether to ignore this rule on merge commits (which typically only have a title)
+# default = True
+ignore-merge-commits=false
+
+[body-changed-file-mention]
+# List of files that need to be explicitly mentioned in the body when they are changed
+# This is useful for when developers often erroneously edit certain files or git submodules.
+# By specifying this rule, developers can only change the file when they explicitly reference
+# it in the commit message.
+#files=gitlint/rules.py,README.md

.uncrustify.cfg

@@ -53,7 +53,6 @@ sp_arith            = add
 sp_bool             = add
 sp_compare          = add
 sp_assign           = add
-sp_after_comma      = add
 sp_func_def_paren   = remove    # "int foo (){" vs "int foo(){"
 sp_func_call_paren  = remove    # "foo (" vs "foo("
 sp_func_proto_paren = remove    # "int foo ();" vs "int foo();"
@@ -93,3 +92,10 @@ nl_after_func_body                       = 2        # number
 
 # The number of newlines after '}' of a single line function body
 nl_after_func_body_one_liner             = 2        # number
+
+indent_extern                            = false
+code_width                               = 120
+cmt_width                                = 120
+sp_after_comma                           = force
+sp_before_comma                          = remove
+nl_max                                   = 2

scripts/git_hooks/commit-msg

@@ -0,0 +1,16 @@
+#!/bin/sh
+
+link_hook()
+{
+    hook_name=$1
+    if test -f "$ZEPHYR_BASE/../sidewalk/.git/hooks/$hook_name"; then
+        rm -rf $ZEPHYR_BASE/../sidewalk/.git/hooks/$hook_name
+    fi
+
+    ln -s $ZEPHYR_BASE/../sidewalk/scripts/git_hooks/$hook_name  $ZEPHYR_BASE/../sidewalk/.git/hooks/$hook_name
+    chmod +x $ZEPHYR_BASE/../sidewalk/scripts/git_hooks/$hook_name
+}
+
+link_hook commit-msg
+link_hook pre-commit
+link_hook prepare-commit-msg

scripts/git_hooks/install_hooks.sh

@@ -3,11 +3,15 @@ UNCRUST_CONFIG="$ZEPHYR_BASE/../sidewalk/.uncrustify.cfg"
 # get list of staged files
 staged_files=$(git diff --name-only --staged --diff-filter=AMR | grep -E '\.(c|h|cpp|hpp)$')
 if [ -n "$staged_files" ]; then
-    echo "## Executing uncrustify"
-    uncrustify -c $UNCRUST_CONFIG --check $staged_files 
-    if [ $? -ne 0 ]; then
-        uncrustify -c $UNCRUST_CONFIG --replace --no-backup $staged_files 
-        echo "suggested changes were made on the tracked files, please review them and stage for commit"
-        exit 1
+    non_mamazon_files=$(grep -l -L "Copyright .* Amazon.com, Inc. or its affiliates. All rights reserved." $staged_files)
+    if [ -n "$non_mamazon_files" ]; then
+
+        echo "## Executing uncrustify"
+        uncrustify -c $UNCRUST_CONFIG --check $non_mamazon_files 
+        if [ $? -ne 0 ]; then
+            uncrustify -c $UNCRUST_CONFIG --replace --no-backup $staged_files 
+            echo "suggested changes were made on the tracked files, please review them and stage for commit"
+            exit 1
+        fi
     fi
 fi