From d166df67f3b4d77d2f54a21b88729b329e07ebb1 Mon Sep 17 00:00:00 2001 From: Jonathan Brouwer Date: Sat, 27 Jun 2026 13:47:14 +0200 Subject: [PATCH 1/2] Sort perf builds in the same order as the PR description --- site/src/github.rs | 9 ++------- site/src/request_handlers/github.rs | 1 - 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/site/src/github.rs b/site/src/github.rs index 5a2d38de1..9aa2e1388 100644 --- a/site/src/github.rs +++ b/site/src/github.rs @@ -40,13 +40,8 @@ pub async fn unroll_rollup( // Sort rolled up commits by their PR number in ascending order, so that they have the // same ordering as in the rollup PR description. - let mut unrolled_builds: Vec = - enqueue_unrolled_try_builds(&gh_client, rollup_merges, previous_master).await?; - // The number should really be an integer, but if not, we will just sort the "non-integer" PRs - // first. - unrolled_builds.sort_by_cached_key(|commit| commit.original_pr_number.parse::().ok()); - - let mapping = unrolled_builds + let mapping = enqueue_unrolled_try_builds(&gh_client, rollup_merges, previous_master) + .await? .into_iter() .fold(String::new(), |mut string, c| { use std::fmt::Write; diff --git a/site/src/request_handlers/github.rs b/site/src/request_handlers/github.rs index 07cdf5a9f..a8b4b98f2 100644 --- a/site/src/request_handlers/github.rs +++ b/site/src/request_handlers/github.rs @@ -52,7 +52,6 @@ async fn handle_push(ctxt: Arc, push: github::Push) -> ServerResult Date: Mon, 29 Jun 2026 13:43:07 +0200 Subject: [PATCH 2/2] Remove comment --- site/src/github.rs | 2 -- 1 file changed, 2 deletions(-) diff --git a/site/src/github.rs b/site/src/github.rs index 9aa2e1388..8ce5aa438 100644 --- a/site/src/github.rs +++ b/site/src/github.rs @@ -38,8 +38,6 @@ pub async fn unroll_rollup( format!("[{display}]({})", commit_link(s)) }; - // Sort rolled up commits by their PR number in ascending order, so that they have the - // same ordering as in the rollup PR description. let mapping = enqueue_unrolled_try_builds(&gh_client, rollup_merges, previous_master) .await? .into_iter()