Repository navigation
Contribution Report, October 2015 #3728
Description
Activity
Do we get any rewards? Maybe a badge, or a sticker?
@indutny you get a high-five :)
I have made 1 commits.
But you have to high five yourself.
On Nov 9, 2015 5:48 PM, "Mikeal Rogers" notifications@github.com wrote:@indutny https://github.com/indutny you get a high-five :)
—
Reply to this email directly or view it on GitHub
#3728 (comment).- This is awesome, BTW.
How is the reviewers section calculated? If it's from the PR metadata, I don't think it's correct.
Where does "merge masters" come from?
I kinda like it. Master's of the Merge!
@cjihrig it's pulled from the "Reviewed-By" commit message lines.
@thealphanerd "Merge Masters" are people who merged code that they did not author :)
@mikeal I haven't merged anything, AFAIK
it's pulled from the "Reviewed-By" commit message lines.
It's definitely not correct then. I'm not listed as a reviewer, but here is one from a few days ago. 33827e3
@cjihrig @mikeal ... depending on how those Reviewed-By commit messages are pulled, they may not catch all of them. @rvagg's pr-metadata tool misses quite a few legitimate LGTM's and some get missed here and there when the commit is being landed (particularly PRs will long threads). I have a todo item on my list to take a pass on improving the detection.
There's a small issue with the way we are calculating "new contributor." I'm using the statistics API to pull the weekly data but it appears to not be complete. It works most of the time but for people who have very sparse activity they aren't showing up.
^ updated all the stats, we were excluding cases where we didn't have a consistent email/name combination.
4 remaining items
@srl295 that's because your github metadata is "srl@icu-project.org" and since all your commits are signed that way we don't have the info to map this other email address to that commit.
@jasnell we could pretty easily, but wouldn't it be better to post to the LTS WG these stats about specific branches rather than just adding data to this?
Yeah, that works too. Any plans on having your code generate these stats automatically for the website? ;-)
- addedmetaIssues and PRs related to the general management of the project.Issues and PRs related to the general management of the project.
on Nov 10, 2015 @mikeal hm - https://github.com/srl295 shows the other address (is that what it's using?)
also is
gmai.coma typo?@jasnell it's pretty intense to run the numbers across every repo, especially if I fix the "new contributor" logic to pull the actual commit history, so probably not. It's certainly something I can automate on a month basis though.
@srl295 yup, but locally you sign your commits with the other email address, so in the commit metadata that's the only reference we have. There isn't a good way to ask github if an email address has a login associated, you can only ask what email address is associated with a specific login.
@srl295
gmai.comis a typo someone wrote in one of the review-by comments.@mikeal got it. sorry for making a mess.. trying to keep things consistent from here
💃 for all!
@mikeal couldn't you just refer to the .mailmap for different emails?
If they aren't there already they should be.
jyan@ca.ibm.ca == @john-yan :)
@Fishrock123 the .mailmap is nowhere near a full representation, there's more data just looking at the login associated with each email in a month of commits :/
Closing this. We now have an issue opened for new committers based on this #3761
I wrote some code that can output reports on repo activity. This report is from October 1st to now, it shows how many contributors we've had, which are new since the beginning of October, and if they are collaborators. This should help us identify new people to onboard sooner.
Top Level Stats
Contributors since 2015-10-01T00:00:00Z
Reviewers
Merge masters