Merge pull request #1 from protocol/improve-bash-piping

marten-seemann · web-flow · commit 344272cee55a · 2021-05-13T14:35:34.000-07:00
don't set bash options, use pipes instead
diff --git a/action.yml b/action.yml
@@ -15,12 +15,10 @@ runs:
         for dir in $(find . \( -name vendor -o -name '[._].*' -o -name node_modules \) -prune -o -name go.mod -print | sed 's:/go.mod$::'); do
           echo "#=#=#=# $dir #=#=#=#"
           pushd $dir > /dev/null
-          set +e -x
           ( 
             ${{ inputs.run }}
           )
           s=$?
-          set -e +x
           if [[ $s != 0 ]]; then status=$s; fi
           popd > /dev/null
           echo -e "\n\n"

Original file line number	Diff line number	Diff line change
`@@ -15,12 +15,10 @@ runs:`
`15`	`15`	`for dir in $(find . \( -name vendor -o -name '[._].*' -o -name node_modules \) -prune -o -name go.mod -print \| sed 's:/go.mod$::'); do`
`16`	`16`	`echo "#=#=#=# $dir #=#=#=#"`
`17`	`17`	`pushd $dir > /dev/null`
`18`		`- set +e -x`
`19`	`18`	`(`
`20`	`19`	`${{ inputs.run }}`
`21`	`20`	`)`
`22`	`21`	`s=$?`
`23`		`- set -e +x`
`24`	`22`	`if [[ $s != 0 ]]; then status=$s; fi`
`25`	`23`	`popd > /dev/null`
`26`	`24`	`echo -e "\n\n"`