[ci] skip doc jobs - circleCI is not reliable - disable skip for now (#8926)

* disable skipping, but leave logging for the future

[ci] skip doc jobs - circleCI is not reliable - disable skip for now (#8926)
* disable skipping, but leave logging for the future
73c51f7f · Stas Bekman · GitHub · 71688a88 · 73c51f7f
Unverified Commit 73c51f7f authored Dec 04, 2020 by Stas Bekman Committed by GitHub Dec 04, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

.circleci/config.yml .circleci/config.yml +3 -1

No files found.
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -14,12 +14,14 @@ commands:
            # pipeline.git.base_revision is not always defined, so only proceed if all external vars are defined
            if test -n "<< pipeline.git.base_revision >>" && test -n "<< pipeline.git.revision >>" && test -n "$(git diff --name-only << pipeline.git.base_revision >>...<< pipeline.git.revision >>)"
            then
+                git diff --name-only << pipeline.git.base_revision >>...<< pipeline.git.revision >> 
                if git diff --name-only << pipeline.git.base_revision >>...<< pipeline.git.revision >> | egrep -qv '\.(md|rst)$'
                then
                    echo "Non-docs were modified in this PR, proceeding normally"
                else
                    echo "Only docs were modified in this PR, quitting this job"
-                    circleci step halt
+                    # disable skipping for now, as circleCI's base_revision is inconsistent leading to invalid ranges
+                    # circleci step halt
                fi
            else
                echo "Can't perform skipping check w/o base_revision defined, continuing the job"