Update PRESUBMIT.cfg to include checks for release_notes.
BUG=b/173034990
TEST=manually checking on some changelists.
RELEASE_NOTE=None
Change-Id: I0513f12a5d6679ac4a68d4c6cbdd2239c300cd34
Reviewed-on: https://cos-review.googlesource.com/c/cos/overlays/board-overlays/+/8561
Tested-by: Vaibhav Rustagi <vaibhavrustagi@google.com>
Reviewed-by: Dexter Rivera <riverade@google.com>
diff --git a/PRESUBMIT.cfg b/PRESUBMIT.cfg
index 88eaf2e..1038191 100644
--- a/PRESUBMIT.cfg
+++ b/PRESUBMIT.cfg
@@ -1,11 +1,13 @@
[Hook Scripts]
cros lint = ../../chromite/bin/cros lint ${PRESUBMIT_FILES}
-hook0 = project-lakitu/presubmit_check.sh
[Hook Overrides]
# Make sure Manifest files are valid.
manifest_check: true
+# Make sure RELEASE_NOTE field is present.
+release_note_field_check: true
+
# On by default, but required for options below.
cros_license_check: false
cos_ebuild_license_header_check: true
diff --git a/overlay-lakitu/presubmit_check.sh b/overlay-lakitu/presubmit_check.sh
deleted file mode 100755
index 9440bb0..0000000
--- a/overlay-lakitu/presubmit_check.sh
+++ /dev/null
@@ -1,23 +0,0 @@
-#!/bin/bash
-#
-# Copyright 2017 The Chromium OS Authors. All rights reserved.
-# Use of this source code is governed by a BSD-style license that can be
-# found in the LICENSE file.
-
-readonly lakitu_files=$(echo "${PRESUBMIT_FILES}" | grep overlay-lakitu/)
-
-if [[ -z "${lakitu_files}" ]]; then
- exit 0
-fi
-
-readonly commit_desc="$(git log --format=%s%n%n%b "${PRESUBMIT_COMMIT}^!")"
-
-# Checks for a non empty RELEASE_NOTE= filed.
-RELEASE_NOTE_RE='RELEASE_NOTE=[^[:blank:]]+'
-if [[ ! "${commit_desc}" =~ ${RELEASE_NOTE_RE} ]]; then
- echo "Changelist description needs a RELEASE_NOTE= field (No space after =):"
- echo "RELEASE_NOTE=New feature"
- echo "RELEASE_NOTE=Bugfix"
- echo "RELEASE_NOTE=None"
- exit 1
-fi
