forked from the-distro/ofborg
tasks/eval/nixpkgs: don't request review from PR author
GitHub doesn't let us, so we silently skip them.
This commit is contained in:
parent
8e284c5b6b
commit
5fd48be23d
|
@ -566,8 +566,18 @@ impl<'a> EvaluationStrategy for NixpkgsStrategy<'a> {
|
||||||
}
|
}
|
||||||
|
|
||||||
fn request_reviews(maint: &maintainers::ImpactedMaintainers, pull: &hubcaps::pulls::PullRequest) {
|
fn request_reviews(maint: &maintainers::ImpactedMaintainers, pull: &hubcaps::pulls::PullRequest) {
|
||||||
|
let pull_meta = pull.get();
|
||||||
|
|
||||||
if maint.maintainers().len() < 10 {
|
if maint.maintainers().len() < 10 {
|
||||||
for maintainer in maint.maintainers() {
|
for maintainer in maint.maintainers() {
|
||||||
|
if let Ok(meta) = &pull_meta {
|
||||||
|
// GitHub doesn't let us request a review from the PR author, so
|
||||||
|
// we silently skip them.
|
||||||
|
if meta.user.login.to_ascii_lowercase() == maintainer.to_ascii_lowercase() {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
if let Err(e) =
|
if let Err(e) =
|
||||||
pull.review_requests()
|
pull.review_requests()
|
||||||
.create(&hubcaps::review_requests::ReviewRequestOptions {
|
.create(&hubcaps::review_requests::ReviewRequestOptions {
|
||||||
|
|
Loading…
Reference in a new issue