From b7750b588c1d7ae9ea3891a254a39de5d3b3c572 Mon Sep 17 00:00:00 2001 From: Gil Date: Wed, 7 Mar 2018 09:59:38 -0800 Subject: Speed up source checks in Travis (#885) * Factor out a whitespace checking script * Factor out a copyright checking script * Rewrite lint.sh to honor revision ranges * Only restyle files that are part of the PR * Run C++ lint before builds to fail faster * Git grep doesn't operate on revision ranges * style.sh accepts clang-format 6 or 7 --- scripts/check_whitespace.sh | 33 +++++++++++++++++++++++++++++++++ 1 file changed, 33 insertions(+) create mode 100755 scripts/check_whitespace.sh (limited to 'scripts/check_whitespace.sh') diff --git a/scripts/check_whitespace.sh b/scripts/check_whitespace.sh new file mode 100755 index 0000000..a38d4ce --- /dev/null +++ b/scripts/check_whitespace.sh @@ -0,0 +1,33 @@ +# Copyright 2018 Google +# +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. + +# Fail on an trailing whitespace characters, excluding +# * binary files (-I) +# * nanopb-generated files +# +# Note: specifying revisions we care about makes this go slower than just +# grepping through the whole repo. +options=( + -n # show line numbers + -I # exclude binary files + ' $' +) + +git grep "${options[@]}" \ + -- ':(exclude)Firestore/Protos/nanopb' +if [[ $? == 0 ]]; then + echo "ERROR: Trailing whitespace found in the files above. Please fix." + exit 1 +fi + -- cgit v1.2.3