diff options
author | Pietro Albini <pietro@pietroalbini.org> | 2019-04-07 22:23:14 +0200 |
---|---|---|
committer | Pietro Albini <pietro@pietroalbini.org> | 2019-04-07 22:23:14 +0200 |
commit | 9fd6512d2b87eb37547fe3e26b1da0a3c3568031 (patch) | |
tree | 73e88865538a13d9aa463ba101431ec342d6f9f6 /src/validate.rs | |
parent | 8bfc13aab1049fa7904c026aae4038622eb1b19f (diff) |
add ability to exclude team members from rfcbot
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>, |