From 09411f899363d56112cbc885ba74078edfaf8978 Mon Sep 17 00:00:00 2001 From: Chris Smith Date: Thu, 1 Jun 2023 15:02:28 +0100 Subject: [PATCH] Danger: don't complain about reviewers (#509) Now we have a CODEOWNERS file a reviewer is assigned automatically, so there's no need for this. It also misfires a bunch (maybe if there are no _pending_ reviewers because they've already reviewed?), so removing it fixes that noise. --- tools/danger/dangerfile.js | 5 ----- 1 file changed, 5 deletions(-) diff --git a/tools/danger/dangerfile.js b/tools/danger/dangerfile.js index 88e721febb..263c394e25 100644 --- a/tools/danger/dangerfile.js +++ b/tools/danger/dangerfile.js @@ -171,11 +171,6 @@ if (hasPngs) { warn("You seem to have made changes to some images. Please consider using an vector drawable.") } -// Check for reviewers -if (github.requested_reviewers.users.length == 0 && !pr.draft) { - warn("Please add a reviewer to your PR.") -} - // Check that translations have not been modified by developers const translationAllowList = [ "RiotTranslateBot",