Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consider committer data, not just commit author #62

Open
smola opened this issue Sep 7, 2019 · 3 comments
Open

Consider committer data, not just commit author #62

smola opened this issue Sep 7, 2019 · 3 comments
Labels
enhancement New feature or request

Comments

@smola
Copy link

smola commented Sep 7, 2019

A commit has a commit author and committer. They may be the same or not. It seems we are currently using only author data and not committer.

@smola smola added the enhancement New feature or request label Sep 7, 2019
@vmarkovtsev
Copy link
Collaborator

@irinakhismatullina please check this: what do we use to build the graph?

@irinakhismatullina
Copy link
Contributor

Yes, we definitely use commit authors, not committers.

@irinakhismatullina
Copy link
Contributor

I guess it depends on which of them we want to analyze. I don't know very well how big is the difference between the two in practice, but it seems that in theory we are more interested with the authors of the code. And also I'd expect that there were more bots among committers than among authors, but I'm not sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants