Skip to content

Commit

Permalink
Use proper field separator for parsing available CLRs
Browse files Browse the repository at this point in the history
The list of available CLRers in the `$available` variable are
space-separated. So the internal field separator should be set
to space instead of comma.

Plus a minor fix for empty-checking the `$availableCLR`.
  • Loading branch information
junpataleta committed Nov 14, 2024
1 parent 175395f commit 80c4acc
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions tracker_automations/component_leads_integration_mover/lib.sh
Original file line number Diff line number Diff line change
Expand Up @@ -172,14 +172,14 @@ function verify_revievers_availability() {
return # Outcome set, we are done.
fi

# There is some component lead reviewer available, let's send the issue to CLR. This check is done.
# There is at least one component lead reviewer available, let's send the issue to CLR. This check is done.
echo " - There are available reviewers (${available}) for the issue."
outcome=CLR
availableProfiles=()
IFS=, read -r -a availableCLRs <<<"$available"
IFS=' ' read -r -a availableCLRs <<<"$available"
for availableCLR in "${availableCLRs[@]}"; do
availableCLR=$(trimstring "$availableCLR")
if [[ -z ${available} ]]; then
if [[ -z ${availableCLR} ]]; then
continue
fi
availableProfiles+=("[~${availableCLR}]")
Expand Down

0 comments on commit 80c4acc

Please sign in to comment.