You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/05/20 16:48:14 UTC

[GitHub] [airflow] ashb commented on a change in pull request #8924: Re-run all tests when Dockerfile or Github worflow files change

ashb commented on a change in pull request #8924:
URL: https://github.com/apache/airflow/pull/8924#discussion_r428160892



##########
File path: scripts/ci/ci_count_changed_files.sh
##########
@@ -42,7 +42,7 @@ echo
 echo
 echo "Changed files matching the ${2} pattern"
 echo
-echo "${CHANGED_FILES}" | grep "${2}" || true
+echo "${CHANGED_FILES}" | grep -E "${2}" || true
 echo
 
 echo

Review comment:
       > FYI. @ashb It's enough to set VERBOSE_COMMANDS="true" in the workflow and all commands will be printed.
   
   Oh yes, I thought this was deeper in the call stack, so I thought that would have given me a lot more logs than I want. (Plus sometimes the more direct option is easier to remember)




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org