request reviews one at a time
if a batch of review requests contains any users who are not members of the organization, the entire request fails.
This commit is contained in:
parent
22e7d0ae8d
commit
f151c71ebb
|
@ -737,18 +737,16 @@ fn request_reviews(
|
||||||
) {
|
) {
|
||||||
if let Ok(ref maint) = m {
|
if let Ok(ref maint) = m {
|
||||||
if maint.maintainers().len() < 10 {
|
if maint.maintainers().len() < 10 {
|
||||||
|
for maintainer in maint.maintainers() {
|
||||||
if let Err(e) =
|
if let Err(e) =
|
||||||
pull.review_requests()
|
pull.review_requests()
|
||||||
.create(&hubcaps::review_requests::ReviewRequestOptions {
|
.create(&hubcaps::review_requests::ReviewRequestOptions {
|
||||||
reviewers: maint.maintainers(),
|
reviewers: vec![maintainer.clone()],
|
||||||
team_reviewers: vec![],
|
team_reviewers: vec![],
|
||||||
})
|
})
|
||||||
{
|
{
|
||||||
println!(
|
println!("Failure requesting a review from {}: {:#?}", maintainer, e,);
|
||||||
"Failure adding review requests for maintainers ({:?}): {:#?}",
|
}
|
||||||
maint.maintainers(),
|
|
||||||
e,
|
|
||||||
);
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue