-
Notifications
You must be signed in to change notification settings - Fork 196
workaround the case of a non-public email #78
Conversation
👍 |
Uncomment when github/maven-plugins#78 has been merged.
Uncomment when github/maven-plugins#78 has been merged.
Uncomment when github/maven-plugins#78 has been merged.
Uncomment when github/maven-plugins#78 has been merged.
Uncomment when github/maven-plugins#78 has been merged.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this use user.getEmail()
when available and only fall back to userService.getEmails().get(0)
when the first is null/empty?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that might work as well; you'd have to give it a try.
I don't know the specifics of of the egit api anymore and don't have time atm.
workaround the case of a non-public email
Thanks for this, apologies for the delay |
Released 0.11 to Maven Central just now, thanks again. |
Thanks for this !! |
Yeah, I think it still takes hours to sync to maven central. You can use https://oss.sonatype.org/#nexus-search;quick~github-maven-plugins in the interim if you need. |
Uncomment when github/maven-plugins#78 has been merged.
Uncomment when github/maven-plugins#78 has been merged.
As proposed by some others, this works around the issue of not having a public email address on your github profile.
close #69
close #70
close #71
close #73
close #77