Skip to content

Commit cc204a5

Browse files
committed
Ignore self-added Reviewed/Complete labels
1 parent 004e064 commit cc204a5

1 file changed

Lines changed: 13 additions & 9 deletions

File tree

src/prs.rs

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -329,26 +329,30 @@ impl PrMetrics {
329329
pr: Pr,
330330
created_at: chrono::DateTime<chrono::Utc>,
331331
label_add_events: Vec<LabelAddEvent>,
332+
staff: &BTreeSet<GithubLogin>,
332333
) -> PrMetrics {
333334
let mut first_needs_review = None;
334335
let mut first_reviewed = None;
335336
let mut first_complete = None;
336337
let mut iterations = 0;
338+
let mut reviewed_by = ReviewedBy::NoOne;
337339

338340
for event in &label_add_events {
339341
if event.label == "Needs Review" {
340342
if first_needs_review.is_none() {
341343
first_needs_review = Some(event.time);
342344
}
343-
} else if event.label == "Reviewed" {
344-
iterations += 1;
345-
if first_reviewed.is_none() {
346-
first_reviewed = Some(event.time);
347-
}
348-
} else if event.label == "Complete" {
349-
iterations += 1;
350-
if first_complete.is_none() {
351-
first_complete = Some(event.time);
345+
} else if event.actor != pr.author {
346+
if event.label == "Reviewed" {
347+
iterations += 1;
348+
if first_reviewed.is_none() {
349+
first_reviewed = Some(event.time);
350+
}
351+
} else if event.label == "Complete" {
352+
iterations += 1;
353+
if first_complete.is_none() {
354+
first_complete = Some(event.time);
355+
}
352356
}
353357
}
354358
}

0 commit comments

Comments
 (0)