Skip to content

Commit

Permalink
MINOR: [Dev] Fix handling minor issues on Dev PR worflow (apache#44987)
Browse files Browse the repository at this point in the history
### Rationale for this change

This is a follow up from: apache#44975 (comment)

### 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 <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
  • Loading branch information
raulcd authored Dec 10, 2024
1 parent b907c5d commit 9d57a25
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/dev_pr/issue_check.js
Original file line number Diff line number Diff line change
Expand Up @@ -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);
}
};

0 comments on commit 9d57a25

Please sign in to comment.