Map users with the same email to one person about rails-contributors HOT 4 CLOSED

This could be a good idea to explore indeed.

I guess it would go like this: if Commit#extract_candidates decides to fallback to the commit author, and the email is already present in the commits table, return the canonical name assigned to that one.

The only gotcha that comes to mind is that a few emails like "[email protected]" map to two contributors. But these are known so maybe we could workaround that.

Would you like to work on it?

from rails-contributors.

Actually it might be more complicated. The application does not know which heuristic was applied to the existing commit. For example, the commit could have a hard-coded rule to be associated to someone else, or the commit message could have overriden the author, or the commit could have several authors associated. In those cases this check won't yield a correct association.

from rails-contributors.

I have seen more counterexamples.

For example the email "[email protected]" has as authors "Akira Matsuda", "Akira Matsuda + Koichi Sasada", "Akira Matsuda & Yukiko Kawamoto", "Akira Matsuda & Kohei Suzuki", "Akira Matsuda & saya", "Akira Matsuda & Naoto Koshikawa". The email "[email protected]" maps to "Sean Griffin", and also to "@schneems and @sgrif".

There are quite a few examples of email addresses from which you cannot automatically assign a canonical name.

I think this is trickier that it seems, by now the app is going to keep ignoring emails and work only based on names.

from rails-contributors.

Fair enough, thanks for looking into it.

from rails-contributors.