Skip to content

Commit

Permalink
Add failfast to embedded shell scripts
Browse files Browse the repository at this point in the history
  • Loading branch information
anthonydahanne committed Sep 27, 2023
1 parent dc7110d commit 432e52a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions .github/workflows/test-all-samples.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ jobs:
- name: Discover changed paths
id: paths
run: |
set -eo pipefail
all_java_smoke_test_paths=$( find ./java -iname "smoke_test" | jq --raw-input | jq --slurp --compact-output )
all_other_smoke_test_paths='["ca-certificates","dotnet-core","git","go","nodejs","php","procfile","python","ruby","web-servers"]'
all_paths=$( echo -e "$all_java_smoke_test_paths\n$all_other_smoke_test_paths" | jq -s 'add' -c )
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/test-pull-request-java.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ jobs:
- name: Discover changed paths
id: paths
run: |
set -eo pipefail
# 1st step: we get the list of files changed between the tip of this PR branch and main
# Since we're only interested in the folders containing those files, we get them, sort them alphabetically (`sort`), de duplicate them (`uniq`) and filter out dot folders (.github for example)
# we end up with, for example, changed_java_folders="java/akka/smoke_test\njava/kotlin/gradle/wrapper\njava/application-insights"
Expand Down

0 comments on commit 432e52a

Please sign in to comment.