From 3a8eaefa80680fad1b18adb32c9cb0ffc3c62abc Mon Sep 17 00:00:00 2001
From: Wojciech Knapik <1129003+wknapik@users.noreply.github.com>
Date: Tue, 7 Mar 2023 14:20:48 +0100
Subject: [PATCH] Run CI on draft PRs by default (#17430)

* Run CI on draft PRs by default

* WIP

* WIP

* WIP

* WIP

* WIP

* WIP
---
 .github/PULL_REQUEST_TEMPLATE.md | 1 -
 Jenkinsfile                      | 4 ++--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
index d76a154e95a2..e6a1c3edc0ba 100644
--- a/.github/PULL_REQUEST_TEMPLATE.md
+++ b/.github/PULL_REQUEST_TEMPLATE.md
@@ -3,7 +3,6 @@ Resolves
 
 <!-- CI-related labels that can be applied to this PR:
 * CI/run-audit-deps (1) - check for known npm/cargo vulnerabilities (audit_deps)
-* CI/run-draft - run CI builds on a draft PR (otherwise only on non-draft PRs)
 * CI/run-network-audit (1) - run network-audit
 * CI/run-upstream-tests - run Chromium unit and browser tests on Linux and Windows (otherwise only on Linux)
 * CI/skip - do not run CI builds (except noplatform)
diff --git a/Jenkinsfile b/Jenkinsfile
index 992a6aecdc0a..fe02a85ff99f 100644
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -29,7 +29,7 @@ pipeline {
                         GITHUB_AUTH_HEADERS = [[name: 'Authorization', value: 'token ' + PR_BUILDER_TOKEN]]
                         CHANGE_BRANCH_ENCODED = java.net.URLEncoder.encode(CHANGE_BRANCH, 'UTF-8')
                         def prDetails = readJSON(text: httpRequest(url: GITHUB_API + '/brave-core/pulls?head=brave:' + CHANGE_BRANCH_ENCODED, customHeaders: GITHUB_AUTH_HEADERS, quiet: true).content)[0]
-                        SKIP = (prDetails.draft.equals(true) && prDetails.labels.count { label -> label.name.equalsIgnoreCase('CI/run-draft') }.equals(0)) || prDetails.labels.count { label -> label.name.equalsIgnoreCase('CI/skip') }.equals(1) || prDetails.labels.count { label -> label.name.equalsIgnoreCase("CI/skip-${PLATFORM}") }.equals(1)
+                        SKIP = prDetails.labels.count { label -> label.name.equalsIgnoreCase('CI/skip') }.equals(1) || prDetails.labels.count { label -> label.name.equalsIgnoreCase("CI/skip-${PLATFORM}") }.equals(1)
                         RUN_NETWORK_AUDIT = prDetails.labels.count { label -> label.name.equalsIgnoreCase('CI/run-network-audit') }.equals(1)
                         RUN_AUDIT_DEPS = prDetails.labels.count { label -> label.name.equalsIgnoreCase('CI/run-audit-deps') }.equals(1)
                         RUN_UPSTREAM_TESTS = prDetails.labels.count { label -> label.name.equalsIgnoreCase('CI/run-upstream-tests') }.equals(1)
@@ -38,7 +38,7 @@ pipeline {
                     }
 
                     if (SKIP && PLATFORM != 'noplatform') {
-                        echo "Aborting build as PRs are either in draft or have a skip label (CI/skip or CI/skip-${PLATFORM})"
+                        echo "Aborting build as PRs have a skip label (CI/skip or CI/skip-${PLATFORM})"
                         currentBuild.result = 'SUCCESS'
                         return
                     }