Skip to content

Commit

Permalink
ci/request-reviews: lowercase handles consistently
Browse files Browse the repository at this point in the history
This script assumed to get lowercased input before, but with the
addition of pinging maintainers that's not necessarily true anymore.

Since the checks for prAuthor and already-reviewed-by already lowercase,
make sure to lowercase the handles in the users array, too.

(cherry picked from commit 213dbf1)
  • Loading branch information
wolfgangwalther committed Jan 4, 2025
1 parent 1b7595a commit 58e9df7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ci/request-reviews/process-reviewers.sh
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ trap 'rm -rf "$tmp"' exit

declare -A users=()
while read -r handle && [[ -n "$handle" ]]; do
users[$handle]=
users[${handle,,}]=
done

# Cannot request a review from the author
Expand Down

0 comments on commit 58e9df7

Please sign in to comment.