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 ee356d6 commit fd28da3
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 fd28da3

Please sign in to comment.