From 73c51f7fcd849f5621c331d9cccb00fc197c8b88 Mon Sep 17 00:00:00 2001 From: Stas Bekman Date: Fri, 4 Dec 2020 10:13:42 -0800 Subject: [PATCH] [ci] skip doc jobs - circleCI is not reliable - disable skip for now (#8926) * disable skipping, but leave logging for the future --- .circleci/config.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.circleci/config.yml b/.circleci/config.yml index fe881970bb..cde8013e80 100644 --- 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"