Skip to content

Commit

Permalink
Require 2/3 majority
Browse files Browse the repository at this point in the history
  • Loading branch information
JakobDegen committed Aug 12, 2023
1 parent 365810a commit 64403de
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/github/nag.rs
Original file line number Diff line number Diff line change
Expand Up @@ -395,7 +395,7 @@ fn evaluate_pendings() -> DashResult<()> {
proposal.id, why));
}

let majority_complete = num_outstanding_reviews < num_complete_reviews;
let majority_complete = num_outstanding_reviews < 2 * num_complete_reviews;

if num_active_concerns == 0 && majority_complete && num_outstanding_reviews < 3 {
// TODO only record the fcp as started if we know that we successfully commented
Expand Down Expand Up @@ -1197,7 +1197,7 @@ impl<'a> RfcBotComment<'a> {
}
}

msg.push_str("\nOnce a majority of reviewers approve (and at most 2 approvals are outstanding), ");
msg.push_str("\nOnce two thirds of reviewers approve (and at most 2 approvals are outstanding), ");
msg.push_str("this will enter its final comment period. ");
msg.push_str("If you spot a major issue that hasn't been raised ");
msg.push_str("at any point in this process, please speak up!\n");
Expand Down

0 comments on commit 64403de

Please sign in to comment.