Skip to content

[INFRA] Link referenced issues in PR description and unify with label pull requests job #10863

Merged
PHILO-HE merged 1 commit intoapache:mainfrom
PHILO-HE:append-issueID
Oct 13, 2025
Merged

[INFRA] Link referenced issues in PR description and unify with label pull requests job #10863
PHILO-HE merged 1 commit intoapache:mainfrom
PHILO-HE:append-issueID

Conversation

@PHILO-HE
Copy link
Member

What changes are proposed in this pull request?

Previously, the referenced issue ID is posted as comment, which can unnecessarily trigger notification to PR subscribers, including authors, reviewers.

How was this patch tested?

Tested in my personal repo.

@github-actions github-actions bot added the INFRA label Oct 10, 2025
@PHILO-HE
Copy link
Member Author

@zhouyuan, could you take a look?

Copy link
Member

@zhouyuan zhouyuan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@zhouyuan
Copy link
Member

@PHILO-HE
The initial logic is following the process from arrow project:
https://github.com/apache/arrow/tree/main/.github/workflows/dev_pr
it should be not noisy to post this as a comment as it will notify only once.

@PHILO-HE
Copy link
Member Author

@PHILO-HE The initial logic is following the process from arrow project: https://github.com/apache/arrow/tree/main/.github/workflows/dev_pr it should be not noisy to post this as a comment as it will notify only once.

Perhaps moving it to the description could help keep the PR page clean. :)

@PHILO-HE PHILO-HE merged commit 05b0389 into apache:main Oct 13, 2025
3 checks passed
@PHILO-HE PHILO-HE deleted the append-issueID branch October 13, 2025 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants