diff options
Diffstat (limited to 'src/validate.rs')
-rw-r--r-- | src/validate.rs | 29 |
1 files changed, 29 insertions, 0 deletions
diff --git a/src/validate.rs b/src/validate.rs index 21570d2..8c33570 100644 --- a/src/validate.rs +++ b/src/validate.rs @@ -19,6 +19,7 @@ static CHECKS: &[fn(&Data, &mut Vec<String>)] = &[ validate_duplicate_permissions, validate_permissions, validate_rfcbot_labels, + validate_rfcbot_exclude_members, ]; pub(crate) fn validate(data: &Data) -> Result<(), Error> { @@ -298,6 +299,34 @@ fn validate_rfcbot_labels(data: &Data, errors: &mut Vec<String>) { }); } +/// Ensure rfcbot's exclude-members only contains not duplicated team members +fn validate_rfcbot_exclude_members(data: &Data, errors: &mut Vec<String>) { + wrapper(data.teams(), errors, move |team, errors| { + if let Some(rfcbot) = team.rfcbot_data() { + let mut exclude = HashSet::new(); + let members = team.members(data)?; + wrapper(rfcbot.exclude_members.iter(), errors, move |member, _| { + if !exclude.insert(member) { + bail!( + "duplicate member in `{}` rfcbot.exclude-members: {}", + team.name(), + member + ); + } + if !members.contains(member.as_str()) { + bail!( + "person `{}` is not a member of team `{}` (in rfcbot.exclude-members)", + member, + team.name() + ); + } + Ok(()) + }); + } + Ok(()) + }); +} + fn wrapper<T, I, F>(iter: I, errors: &mut Vec<String>, mut func: F) where I: Iterator<Item = T>, |