From ed58d790cb7e8465ebb56d8536aa3e7a53ba9dad Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ra=C3=BAl=20Cumplido?= Date: Sun, 28 May 2023 17:01:43 +0200 Subject: [PATCH 1/2] GH-14790: [Dev] Avoid extra comment with Closes XXXX on PRs --- .github/workflows/dev_pr/link.js | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/.github/workflows/dev_pr/link.js b/.github/workflows/dev_pr/link.js index 1fbd0447175..94843bbcc8c 100644 --- a/.github/workflows/dev_pr/link.js +++ b/.github/workflows/dev_pr/link.js @@ -83,22 +83,16 @@ async function commentGitHubURL(github, context, pullRequestNumber, issueID) { // Make the call to ensure issue exists before adding comment const issueInfo = await helpers.getGitHubInfo(github, context, issueID, pullRequestNumber); const message = "* Closes: #" + issueInfo.number - if (await haveComment(github, context, pullRequestNumber, message)) { - return; - } if (issueInfo){ + if (context.payload.pull_request.body.includes(message)){ + return; + } await github.pulls.update({ owner: context.repo.owner, repo: context.repo.repo, pull_number: pullRequestNumber, body: (context.payload.pull_request.body || "") + "\n" + message }); - await github.issues.createComment({ - owner: context.repo.owner, - repo: context.repo.repo, - issue_number: pullRequestNumber, - body: message - }); } } From cbf17953ff149ef4299311e2dfd528766025a61e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ra=C3=BAl=20Cumplido?= Date: Mon, 29 May 2023 11:11:50 +0200 Subject: [PATCH 2/2] testing --- dev/merge_arrow_pr.py | 1 + 1 file changed, 1 insertion(+) diff --git a/dev/merge_arrow_pr.py b/dev/merge_arrow_pr.py index 352befc3289..05c1c037269 100755 --- a/dev/merge_arrow_pr.py +++ b/dev/merge_arrow_pr.py @@ -248,6 +248,7 @@ def current_versions(self): return unreleased_versions def resolve(self, fix_version, comment, pr_body): + # comment cur_status = self.issue["state"] if cur_status == "closed":