From 9d57a252ec7f91a341ae724b51bd1160b9bf6b55 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ra=C3=BAl=20Cumplido?= Date: Tue, 10 Dec 2024 12:56:24 +0100 Subject: [PATCH] MINOR: [Dev] Fix handling minor issues on Dev PR worflow (#44987) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ### Rationale for this change This is a follow up from: https://github.com/apache/arrow/pull/44975#issuecomment-2529683575 ### What changes are included in this PR? Fix the issue_check workflow to not assume all PRs are going to have an associated issue. Fix the worflow for minor issues. ### Are these changes tested? I've tested on my fork. ### Are there any user-facing changes? No Authored-by: Raúl Cumplido Signed-off-by: Sutou Kouhei --- .github/workflows/dev_pr/issue_check.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/dev_pr/issue_check.js b/.github/workflows/dev_pr/issue_check.js index f778f70c0f192..176349cdd0512 100644 --- a/.github/workflows/dev_pr/issue_check.js +++ b/.github/workflows/dev_pr/issue_check.js @@ -78,7 +78,7 @@ module.exports = async ({github, context}) => { const pullRequestNumber = context.payload.number; const title = context.payload.pull_request.title; const issue = helpers.detectIssue(title) - if (issue){ + if (issue && issue.kind === "github") { await verifyGitHubIssue(github, context, pullRequestNumber, issue.id); } };