Skip to content

Commit

Permalink
Fix missing pipe in check_ci (#1926)
Browse files Browse the repository at this point in the history
Bugfix:  left out pipe to word count` | wc -l` after updating:

`num_platforms=$(ls ../platforms/config.* | wc -l)` to
`num_platforms=$(find ../platforms -type f -name "config.*")`

when it was flagged by the BASH linter.
  • Loading branch information
TerrenceMcGuinness-NOAA authored Oct 12, 2023
1 parent 51d93e1 commit c2d112b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ci/scripts/check_ci.sh
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ for pr in ${pr_list}; do
# Check to see if this PR that was opened by the weekly tests and if so close it if it passed on all platforms
weekly_labels=$(${GH} pr view "${pr}" --repo "${REPO_URL}" --json headRefName,labels,author --jq 'select(.author.login | contains("emcbot")) | select(.headRefName | contains("weekly_ci")) | .labels[].name ') || true
if [[ -n "${weekly_labels}" ]]; then
num_platforms=$(find ../platforms -type f -name "config.*")
num_platforms=$(find ../platforms -type f -name "config.*" | wc -l)
passed=0
for platforms in ../platforms/config.*; do
machine=$(basename "${platforms}" | cut -d. -f2)
Expand Down

0 comments on commit c2d112b

Please sign in to comment.