From d4d1e10228e8ec37f203476f298855fc7a412e74 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakub=20Ber=C3=A1nek?= Date: Tue, 4 Nov 2025 18:20:50 +0100 Subject: [PATCH] Unify logic for showing all Project Members and generating individual pages for each person --- src/teams.rs | 24 ++++++++++++++---------- 1 file changed, 14 insertions(+), 10 deletions(-) diff --git a/src/teams.rs b/src/teams.rs index 31a2209e..8f14ae53 100644 --- a/src/teams.rs +++ b/src/teams.rs @@ -209,7 +209,7 @@ impl RustTeams { alumni.extend(team.alumni.iter().map(|m| (m.github_id, m))); } for team in &self.teams { - if team.name != "alumni" { + if is_active_team(team) { alumni.extend(team.alumni.iter().map(|m| (m.github_id, m))); active.extend(team.members.iter().map(|m| (m.github_id, m))); } @@ -278,15 +278,13 @@ impl RustTeams { } } for team in &self.teams { - if team.kind == TeamKind::MarkerTeam && team.website_data.is_none() { - continue; - } - - for member in &team.members { - add_team(&mut people, self, member, team, TeamMode::Member); - } - for member in &team.alumni { - add_team(&mut people, self, member, team, TeamMode::Alumni); + if is_active_team(team) { + for member in &team.members { + add_team(&mut people, self, member, team, TeamMode::Member); + } + for member in &team.alumni { + add_team(&mut people, self, member, team, TeamMode::Alumni); + } } } @@ -322,6 +320,12 @@ impl RustTeams { } } +/// Is this a team for which we should display its members on the "All Project Members" page, +/// and whose members should receive a separate person page? +fn is_active_team(team: &Team) -> bool { + team.name != "alumni" +} + /// Get a relative URL of a team that should be appended to /// Should only be used for top-level teams. fn get_team_relative_url(team: &Team) -> String {