andymeneely / chromium-history

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

Metrics: vulnerability_inspections for Developer and vulnerability_inspectors for Filepath #103

Closed andymeneely closed 10 years ago

andymeneely commented 10 years ago

For a given Developer and a given date, return the number of vulnerability inspections they have inspected leading up to that date.

For a given Filepath and a given date, return the number of vulnerability developers they have inspected leading up to that date.

cketant commented 10 years ago

Created branch vulnerable_inspections for this issue. Creating methods: vulnerability_inspections on Developer and vulnerability_inspectors on Filepath

cketant commented 10 years ago

@andymeneely we are defining the Developer who inspected the vulnerabilities as one of the Reviewers on the Code Review correct?

andymeneely commented 10 years ago

Yes, for now one of the reviewers. We might decide to circle back and redefined it as participant or contributor, but for now reviewers.

cketant commented 10 years ago

reviewers method on Filepath model does not return reviewers but the Filepath

andymeneely commented 10 years ago

Let's change this to participants instead of reviewers. That basically makes this a duplicate of #113.

cketant commented 10 years ago

For vulnerability_inspectors for Filepath are we returning the number of vulnerability reviewers, participants, or developers?

andymeneely commented 10 years ago

Participants

cketant commented 10 years ago

Both methods implemented working on stats and verifies for these now.

cketant commented 10 years ago

Meeting with @bspates to optimize this.