From 819782e73842bbc20bea8957510f8eb58cfd9753 Mon Sep 17 00:00:00 2001 From: Dan Mick Date: Thu, 23 May 2024 16:27:06 -0700 Subject: [PATCH] scripts/build_utils.sh: set DOCS_ONLY for PRs with only admin/ files, too DOCS_ONLY is set if the filenames all start with docs/, but it's also true that if the filenames start with admin/, it's strictly docs related (discovered for https://github.com/ceph/ceph/pull/57649) Signed-off-by: Dan Mick --- scripts/build_utils.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/build_utils.sh b/scripts/build_utils.sh index 090159f3..5bbc4888 100755 --- a/scripts/build_utils.sh +++ b/scripts/build_utils.sh @@ -1705,7 +1705,7 @@ docs_pr_only() { files="$(git diff --name-only origin/${ghprbTargetBranch}...origin/pr/${ghprbPullId}/head)" fi echo -e "changed files:\n$files" - if [ $(echo "$files" | grep -v '^doc/' | wc -l) -gt 0 ]; then + if [ $(echo "$files" | egrep -v '^(doc/|admin/)' | wc -l) -gt 0 ]; then DOCS_ONLY=false else DOCS_ONLY=true -- 2.39.5