ktmud commented on pull request #13260:
URL: https://github.com/apache/superset/pull/13260#issuecomment-782929216
I wonder if it's easier to understand if we use step **output** instead of
**outcome** , considering all future steps will have to check
```bash
for FILE in ${FILES}
do
for REGEX in "${REGEXES[@]}"
do
if [[ "${FILE}" =~ ${REGEX} ]]; then
echo "Detected changes in following file: ${FILE}"
exit 0
fi
done
done
echo "No matching file changes detected"
echo "::set-output name=no_matching_change::true"
```
Then maybe we can also use condition check on the check job itself (`if:
github.event_name == 'pull_request'`) to address what this PR is trying to
solve? I.e. run the file change check only on pull requests, and _skip_ future
tasks only if `steps.check_file_change.output.no_matching_change = true`.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
