Skip to content

Commit

Permalink
Update workflow to solve linter
Browse files Browse the repository at this point in the history
  • Loading branch information
@RonBenMosheHabana
RonBenMosheHabana committed Jan 9, 2025
1 parent 5a2c7a0 commit 4089df1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/trigger_jenkins.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ jobs:
id: tests
run: |

Check failure on line 50 in .github/workflows/trigger_jenkins.yml

View workflow job for this annotation

GitHub Actions / actionlint 

shellcheck reported issue in this script: SC2116:style:2:18: Useless echo? Instead of 'cmd $(echo foo)', just use 'cmd foo'

Check failure on line 50 in .github/workflows/trigger_jenkins.yml

View workflow job for this annotation

GitHub Actions / actionlint 

shellcheck reported issue in this script: SC2086:info:2:25: Double quote to prevent globbing and word splitting

Check failure on line 50 in .github/workflows/trigger_jenkins.yml

View workflow job for this annotation

GitHub Actions / actionlint 

shellcheck reported issue in this script: SC2086:info:2:43: Double quote to prevent globbing and word splitting
test_list=$(yq -oj e .jenkins/test_config.yaml | jq "[.stages[].steps[]]")
echo "tests_list=${test_list}" >> $GITHUB_OUTPUT
echo "tests_list=$(echo ${test_list})" >> $GITHUB_OUTPUT
TestRun:
name: Test / ${{matrix.tests.name}}
needs: [CalculateJobs]
Expand Down

0 comments on commit 4089df1

Please sign in to comment.