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:
[email protected]


Reply via email to