Gate against go formatting
The `make test` target includes go formatting, if there is any unformatted code this will lead to a git diff, which the gates now check for. Signed-off-by: Sean Eagan <seaneagan1@gmail.com> Change-Id: If3d6b8cb11e88d6fd4773ae6c2913288dfec12e2
This commit is contained in:
parent
635a6e3b85
commit
511d7ef22e
4
Makefile
4
Makefile
@ -116,3 +116,7 @@ $(LINTER): $(TOOLBINDIR)
|
||||
|
||||
$(TOOLBINDIR):
|
||||
mkdir -p $(TOOLBINDIR)
|
||||
|
||||
.PHONY: check-git-diff
|
||||
check-git-diff:
|
||||
@./tools/git_diff_check
|
@ -59,6 +59,10 @@
|
||||
make:
|
||||
chdir: "{{ zuul.project.src_dir }}"
|
||||
target: test
|
||||
- name: Check git diff
|
||||
make:
|
||||
chdir: "{{ zuul.project.src_dir }}"
|
||||
target: check-git-diff
|
||||
- name: Build Docker image
|
||||
make:
|
||||
chdir: "{{ zuul.project.src_dir }}"
|
||||
|
26
tools/git_diff_check
Executable file
26
tools/git_diff_check
Executable file
@ -0,0 +1,26 @@
|
||||
#!/bin/bash
|
||||
|
||||
# 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.
|
||||
|
||||
# Update git index
|
||||
git update-index -q --ignore-submodules --refresh
|
||||
|
||||
# Evaluate git files for difference, if found print message and fail.
|
||||
git diff-files --quiet --ignore-submodules
|
||||
if [ $? -ne 0 ]
|
||||
then
|
||||
echo "git diff found modified test cases, please run make test"
|
||||
exit 1
|
||||
fi
|
||||
|
||||
echo "no git diff detected, make target completed successfully"
|
Loading…
Reference in New Issue
Block a user