Skip to content

Commit 53d8a71

Browse files
author
meorphis
committed
fix: sort regressions first when determining worst outcome
1 parent 319cfc9 commit 53d8a71

File tree

4 files changed

+11
-8
lines changed

4 files changed

+11
-8
lines changed

dist/build.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/merge.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/preview.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/outcomes.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -232,9 +232,12 @@ export function categorizeOutcome({
232232
.filter((r): r is Exclude<typeof categoryOutcome, null> => r !== null)
233233
.sort(
234234
(a, b) =>
235-
// sort by severity then rank
235+
// sort by regression status then severity then rank
236+
[true, null, false].indexOf(a.isRegression) -
237+
[true, null, false].indexOf(b.isRegression) ||
236238
conclusions.diagnostic.indexOf(a.severity) -
237-
conclusions.diagnostic.indexOf(b.severity) || a.rank - b.rank,
239+
conclusions.diagnostic.indexOf(b.severity) ||
240+
a.rank - b.rank,
238241
)[0];
239242

240243
return {

0 commit comments

Comments
 (0)