search

andymeneely / chromium-history

Scripts and data related Chromium's history
11 stars 4 forks source link

Write reviewers method in Filepath #58

Closed andymeneely closed 10 years ago

andymeneely commented 10 years ago

For a given filepath, go back over all commits for a given filepath and return a distinct list of reviewers.

We'll elaborate more on this method later, such as limiting by time, participation, etc. Just keep it simple for now.

Write a verify for it on the development data, and make sure our development data has multiple code reviews/commits to a single filepath with the same reviewer to test the distinctiveness of this method.

andymeneely commented 10 years ago

Currently, this one is blocked because of #68 and #53.

andymeneely commented 10 years ago

I took this one on. It uses nested associations, which was really hard to understand but then results in some simple coding.

f29e0c0