Skip to content

Commit 65d0964

Browse files
Apply suggestions from code review
1 parent e1555de commit 65d0964

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

.github/workflows/email-check.yaml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ jobs:
2424
env:
2525
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
2626
run: |
27-
# Use Github GraphQL APIs to get the email associated with the PR author.
27+
# Use Github GraphQL APIs to get the email associated with the PR author because this takes into account the GitHub settings for email privacy.
2828
query='
2929
query($login: String!) {
3030
user(login: $login) {
@@ -37,9 +37,6 @@ jobs:
3737
email=$(gh api graphql -f login="$PR_AUTHOR" -f query="$query" -H "Authorization: Bearer $GITHUB_TOKEN" --jq '.data.user.email')
3838
echo "EMAIL_AUTHOR_GH_UI=$email" >> "$GITHUB_OUTPUT"
3939
40-
# Print the email to the log for debugging purposes.
41-
echo "GitHub user's email: $email"
42-
4340
# Create empty comment file
4441
echo "[]" > comments
4542

0 commit comments

Comments
 (0)