Skip to content

Commit 471cf0b

Browse files
committed
Sort members in repositories in the v1 API
1 parent a81b7a1 commit 471cf0b

File tree

1 file changed

+22
-18
lines changed

1 file changed

+22
-18
lines changed

src/static_api.rs

Lines changed: 22 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ use anyhow::{ensure, Context as _, Error};
44
use indexmap::IndexMap;
55
use log::info;
66
use rust_team_data::v1;
7-
use rust_team_data::v1::BranchProtectionMode;
7+
use rust_team_data::v1::{BranchProtectionMode, RepoMember};
88
use std::collections::HashMap;
99
use std::path::Path;
1010

@@ -124,23 +124,27 @@ impl<'a> Generator<'a> {
124124
teams.sort_by_key(|t| t.name.clone());
125125
teams
126126
},
127-
members: r
128-
.access
129-
.individuals
130-
.iter()
131-
.map(|(name, permission)| {
132-
let permission = match permission {
133-
RepoPermission::Admin => v1::RepoPermission::Admin,
134-
RepoPermission::Write => v1::RepoPermission::Write,
135-
RepoPermission::Maintain => v1::RepoPermission::Maintain,
136-
RepoPermission::Triage => v1::RepoPermission::Triage,
137-
};
138-
v1::RepoMember {
139-
name: name.clone(),
140-
permission,
141-
}
142-
})
143-
.collect(),
127+
members: {
128+
let mut members: Vec<RepoMember> = r
129+
.access
130+
.individuals
131+
.iter()
132+
.map(|(name, permission)| {
133+
let permission = match permission {
134+
RepoPermission::Admin => v1::RepoPermission::Admin,
135+
RepoPermission::Write => v1::RepoPermission::Write,
136+
RepoPermission::Maintain => v1::RepoPermission::Maintain,
137+
RepoPermission::Triage => v1::RepoPermission::Triage,
138+
};
139+
v1::RepoMember {
140+
name: name.clone(),
141+
permission,
142+
}
143+
})
144+
.collect();
145+
members.sort_by_key(|m| m.name.clone());
146+
members
147+
},
144148
branch_protections,
145149
archived,
146150
auto_merge_enabled: !managed_by_bors,

0 commit comments

Comments
 (0)