admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

pr-pull: handle empty string cases

Browse Source 
Co-authored-by: Mike McQuaid <mike@mikemcquaid.com>
This commit is contained in:
Jonathan Chang 2020-06-29 18:55:47 +10:00 committed by GitHub
parent bada8dd759
commit 9cab9b7f39
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Library/Homebrew/utils/github.rb
View File

@ -424,13 +424,13 @@ module GitHub
next if commit.present? && commit != r["commit"]["oid"] next if commit.present? && commit != r["commit"]["oid"]
next unless %w[MEMBER OWNER].include? r["authorAssociation"] next unless %w[MEMBER OWNER].include? r["authorAssociation"]
email = if r["author"]["email"].empty? email = if r["author"]["email"].blank?
"#{r["author"]["databaseId"]}+#{r["author"]["login"]}@users.noreply.github.com" "#{r["author"]["databaseId"]}+#{r["author"]["login"]}@users.noreply.github.com"
else else
r["author"]["email"] r["author"]["email"]
end end
name = r["author"]["name"] || r["author"]["login"] name = r["author"]["name"].presence || r["author"]["login"]
{ {
"email" => email, "email" => email,