From 73da4b721513f3d947746371e20e782603b1eaab Mon Sep 17 00:00:00 2001 From: John Kleinschmidt Date: Mon, 28 Oct 2019 09:40:21 -0400 Subject: [PATCH] build: fix doc only change when there isn't a PR (#20749) * build: fix doc only change when there isn't a PR Fixes issue where CI was mistakenly marking a PR as a doc only change because the CI was kicked off before the PR was created. --- script/doc-only-change.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/script/doc-only-change.js b/script/doc-only-change.js index 498f1a3428..a5569dfea8 100644 --- a/script/doc-only-change.js +++ b/script/doc-only-change.js @@ -20,8 +20,8 @@ async function checkIfDocOnlyChange () { if (prsForBranch.data.length === 1) { pullRequestNumber = prsForBranch.data[0].number } else { - // If there is more than one PR on a branch, just assume that this is more than a doc change - process.exit(0) + // If there are 0 PRs or more than one PR on a branch, just assume that this is more than a doc change + process.exit(1) } } else if (args.prURL) { // CircleCI doesn't provide the PR number for branch builds, but it does provide the PR URL