From e4fdb9124042cb21ecfb3df8004f0697756bbbad Mon Sep 17 00:00:00 2001 From: Aseda Aboagye Date: Tue, 18 Jun 2019 18:12:30 -0700 Subject: util: presubmit_check.sh: Exclude OWNERS file. The OWNERS file isn't used by `make buildall`, so it shouldn't cause a developer to run `make buildall` before submitting a change to the file. This commit simply filters out changes to the OWNERS file from the presubmit_check script. BUG=None BRANCH=None TEST=Modify OWNERS. Try to upload and verify that the presubmit check does not flag it. Change-Id: I71452ac0a751335f815c707b554a2578c532b476 Signed-off-by: Aseda Aboagye Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1666754 Tested-by: Aseda Aboagye Auto-Submit: Aseda Aboagye Reviewed-by: Daisuke Nojiri Commit-Queue: Aseda Aboagye --- util/presubmit_check.sh | 2 ++ 1 file changed, 2 insertions(+) diff --git a/util/presubmit_check.sh b/util/presubmit_check.sh index 306a1a632d..822633f0f8 100755 --- a/util/presubmit_check.sh +++ b/util/presubmit_check.sh @@ -23,6 +23,8 @@ changed=$(echo "${changed}" | grep -v util/flash_ec) changed=$(echo "${changed}" | grep -v util/uart_stress_tester.sh) # Filter out this file itself. changed=$(echo "${changed}" | grep -v util/presubmit_check.sh) +# Filter out the OWNERS file. +changed=$(echo "${changed}" | grep -v OWNERS) if [[ -n "${changed}" ]]; then echo "Files have changed since last time unit tests passed:" echo "${changed}" | sed -e 's/^/ /' -- cgit v1.2.1