]> git.apps.os.sepia.ceph.com Git - ceph-build.git/commitdiff
scripts/build_utils.sh: set DOCS_ONLY for PRs with only admin/ files, too 2244/head
authorDan Mick <dmick@redhat.com>
Thu, 23 May 2024 23:27:06 +0000 (16:27 -0700)
committerDan Mick <dmick@redhat.com>
Thu, 23 May 2024 23:29:20 +0000 (16:29 -0700)
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 <dmick@redhat.com>
scripts/build_utils.sh

index 090159f384d516793878b7ff6073580ca44a4b10..5bbc48888ff436e421521fa7f6ea5ba69f7c2423 100755 (executable)
@@ -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