Attention is currently required from: plaisthos.

Hello plaisthos,

I'd like you to do a code review.
Please visit

    http://gerrit.openvpn.net/c/openvpn/+/1031?usp=email

to review the following change.


Change subject: git-pre-commit-uncrustify: Make sure to not run on branches 
with clang-format
......................................................................

git-pre-commit-uncrustify: Make sure to not run on branches with clang-format

If a .clang-format file is present, do not run this hook.
This allows to keep this installed while working also on
newer branches that do not use uncrustify anymore.

Change-Id: Ifd36651a3713dd6f2e585f0771f349178a494360
Signed-off-by: Frank Lichtenheld <fr...@lichtenheld.com>
---
M dev-tools/git-pre-commit-uncrustify.sh
1 file changed, 6 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.openvpn.net:29418/openvpn refs/changes/31/1031/1

diff --git a/dev-tools/git-pre-commit-uncrustify.sh 
b/dev-tools/git-pre-commit-uncrustify.sh
index 9851c21..1f49086 100755
--- a/dev-tools/git-pre-commit-uncrustify.sh
+++ b/dev-tools/git-pre-commit-uncrustify.sh
@@ -58,6 +58,12 @@
     exit 0
 fi

+
+if [ -f .clang-format ]; then
+    echo "skipping uncrustify due to presence of .clang-format"
+    exit 0
+fi
+
 # check whether the given file matches any of the set extensions
 matches_extension() {
     local filename="$(basename -- "$1")"

--
To view, visit http://gerrit.openvpn.net/c/openvpn/+/1031?usp=email
To unsubscribe, or for help writing mail filters, visit 
http://gerrit.openvpn.net/settings

Gerrit-Project: openvpn
Gerrit-Branch: release/2.6
Gerrit-Change-Id: Ifd36651a3713dd6f2e585f0771f349178a494360
Gerrit-Change-Number: 1031
Gerrit-PatchSet: 1
Gerrit-Owner: flichtenheld <fr...@lichtenheld.com>
Gerrit-Reviewer: plaisthos <arne-open...@rfc2549.org>
Gerrit-CC: openvpn-devel <openvpn-devel@lists.sourceforge.net>
Gerrit-Attention: plaisthos <arne-open...@rfc2549.org>
Gerrit-MessageType: newchange
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to