Skip to content

Commit d65ed4b

Browse files
authored
Merge pull request #1309 from rylev/change-deserves-attention
Change `deserves_attention` algorithm
2 parents 5a14ec4 + 8d28702 commit d65ed4b

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

site/src/comparison.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -359,13 +359,13 @@ pub(crate) fn deserves_attention(
359359
) -> bool {
360360
match (primary.largest_change(), secondary.largest_change()) {
361361
(Some(c), _) if c.magnitude() >= Magnitude::Medium => true,
362-
(_, Some(c)) if c.magnitude() >= Magnitude::Medium => true,
362+
(_, Some(c)) if c.magnitude() >= Magnitude::Large => true,
363363
_ => {
364364
// How we determine whether a group of small changes deserves attention is and always will be arbitrary,
365365
// but this feels good enough for now. We may choose in the future to become more sophisticated about it.
366366
let primary_n = primary.num_changes();
367367
let secondary_n = secondary.num_changes();
368-
(primary_n * 2 + secondary_n) >= 6
368+
(primary_n * 3 + secondary_n) >= 9
369369
}
370370
}
371371
}

0 commit comments

Comments
 (0)